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

Remove QueryFailed #23382

Merged
1 commit merged into from
Nov 18, 2020
Merged

Remove QueryFailed #23382

1 commit merged into from
Nov 18, 2020

Commits on Nov 18, 2020

  1. Remove QueryFailed

    Mainly error in projection binding expression visitor, when we have an unknown extension expression which is hit when client evaluating, (non-client eval we wouldn't reach this code).
    Translation failure seems most appropriate error.
    Returning NoTranslationExpression will actually provide lesser value in what failed exactly.
    
    Cosmos has a lot of places where it used but those are supposed to invariants and shouldn't exist after Cosmos Query pipeline is updated.
    
    Resolves #22508
    smitpatel committed Nov 18, 2020
    Configuration menu
    Copy the full SHA
    0d9a638 View commit details
    Browse the repository at this point in the history