Kadir Ozdemir
Kadir Ozdemir
The PR looks good to me. Can we trigger the pre-checkin tests for this PR? Not sure why it was not triggered automatically yet.
> Just to be clear, based on my understanding of the design (see doc here: https://docs.google.com/document/d/1Ea42tEBh2X2fCq0_tXSe1BgEqBz58oswJULEbA8-MfI/edit ) we can integrate this change into the default store engine without requiring opt...
@richardantal, the PR looks good to me overall. Thank you for fixing this issue. I will approve the PR after a successful build and test run (expect known test failures).
Can you add tests for both varbinary and varbinary encoded to WhereCompilerTest# testWhereInclusion and PartialIndexIT?