Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Dedupe fixes CRM-18442, CRM-18539, CRM-18480 #8251

Merged
merged 3 commits into from
May 19, 2016
Merged

Dedupe fixes CRM-18442, CRM-18539, CRM-18480 #8251

merged 3 commits into from
May 19, 2016

Commits on May 19, 2016

  1. CRM-18480 fix batch merge job to cope with new inability to re-merge … 

    …deleted contacts

Also, I changed the fatal to an exception - low level functions should throw exceptions, it's up to the form to catch & render those
    @eileenmcnaughton
    eileenmcnaughton committed May 19, 2016
    Configuration menu
    Copy the full SHA
    7827275 View commit details
    Browse the repository at this point in the history

  2. CRM-18842 Dedupe query: remove OR join in favour of more performant U… 

    …NION

Unions are much faster than OR joins. This change took the length of the query to get the dedupes on a large database from
'as long as it took for the server to fall over' to less than one second on a small group of contacts

This query is only affecting one path - ie Individuals - at the moment as I can only extend that as fast as I can write tests.
    @eileenmcnaughton
    eileenmcnaughton committed May 19, 2016
    Configuration menu
    Copy the full SHA
    c232f00 View commit details
    Browse the repository at this point in the history

  3. CRM-18539: Permit criteria for batch merging (other than group)

    @eileenmcnaughton
    eileenmcnaughton committed May 19, 2016
    Configuration menu
    Copy the full SHA
    e23e26e View commit details
    Browse the repository at this point in the history