Skip to content

[WV-2618] queryset_find_possible_duplicate_candidates_to_merge_with_this_candidate inefficient query [Team Review]#3058

Merged
DaleMcGrew merged 4 commits intowevote:developfrom
pravingv:WV_2618_canidate_queryset
Apr 16, 2026
Merged

[WV-2618] queryset_find_possible_duplicate_candidates_to_merge_with_this_candidate inefficient query [Team Review]#3058
DaleMcGrew merged 4 commits intowevote:developfrom
pravingv:WV_2618_canidate_queryset

Conversation

@pravingv
Copy link
Copy Markdown
Contributor

@pravingv pravingv commented Mar 22, 2026

What github.com/wevote server/issues does this fix
WV-2618 queryset_find_possible_duplicate_candidates_to_merge_with_this_candidate inefficient query

Changes included this pull request?
candidate/controllers.py
Added trigram-based query support to improve SQL query performance. If trigram indexes are unavailable, the original query is used as a fallback so results are still returned.

**Note: **This PR should be reviewed and executed together with the following PRs:
#3055
— Needed to show trigram vs. non-trigram search results.
#3054
— Generates the required trigram indexes for improved SQL query performance.

@pravingv pravingv changed the title [WV-2618] queryset_find_possible_duplicate_candidates_to_merge_with_this_candidate inefficient query [WV-2618] queryset_find_possible_duplicate_candidates_to_merge_with_this_candidate inefficient query [Team Review] Mar 27, 2026
@mjacquot1
Copy link
Copy Markdown
Contributor

Seems good!

@DaleMcGrew
Copy link
Copy Markdown
Member

👍

@DaleMcGrew DaleMcGrew merged commit 37e7fcf into wevote:develop Apr 16, 2026
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants