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

Fix to #24671 - Relational: No tests for TranslateContains path #25768

Merged
merged 1 commit into from
Sep 1, 2021
Merged

Fix to #24671 - Relational: No tests for TranslateContains path #25768

merged 1 commit into from
Sep 1, 2021

Commits on Sep 1, 2021

  1. Fix to #24671 - Relational: No tests for TranslateContains path 

    All Contains calls are converted to Any with predicate (which allows to handle null semantics) so this code never gets but fixing this tiny bug, just in case. We were not setting the cardinality of the contains translation to single result, so the translation would have failed, if it wasn't converted to Any and therefore (correctly) being handled by it's translation code.

Fixes #24671
    @maumar
    maumar committed Sep 1, 2021
    Configuration menu
    Copy the full SHA
    5c6b726 View commit details
    Browse the repository at this point in the history