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.

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

SQL: Optimisation fixes for conjunction merges #50703

Merged
merged 2 commits into from
Jan 8, 2020

Commits on Jan 7, 2020

  1. SQL: Optimisation fixes for conjunction merges

    This commit fixes the following issues around the way comparisions are
    merged with ranges in conjunctions:
    * the decision to include the equality of the lower limit is corrected;
    * the selection of the upper limit is corrected to use the upper bound
    of the range;
    * the list of terms in the conjunction is sorted to have the ranges at
    the bottom; this allows subsequent binary comarisions to find compatible
    ranges and potentially be merged away. The end guarantee being that the
    optimisation takes place irrespective of the order of the conjunction
    terms in the statement.
    
    Some comments are also corrected.
    bpintea committed Jan 7, 2020
    Configuration menu
    Copy the full SHA
    f8a73cd View commit details
    Browse the repository at this point in the history

Commits on Jan 8, 2020

  1. adress review observation on anon. comparator

    Replace anonymous comparator of split AND Expressions with a lambda.
    bpintea committed Jan 8, 2020
    Configuration menu
    Copy the full SHA
    9587bfe View commit details
    Browse the repository at this point in the history