mihaibudiu commented on code in PR #4816:
URL: https://github.com/apache/calcite/pull/4816#discussion_r2875547621
##########
testkit/src/main/java/org/apache/calcite/sql/parser/SqlParserTest.java:
##########
@@ -1410,6 +1410,32 @@ private void checkLarge(int n) {
.ok("VALUES (ROW((`A` BETWEEN ASYMMETRIC ((`B` OR (`C` AND `D`)) OR
`E`) AND `F`)))");
}
+ @Test void testBetweenInverse() throws Exception {
Review Comment:
It is always used. it's a part of the SqlToRelConverter, you have to replace
it.
`ConfigBuilder` has field to set the `convertletTable`, but it does not seem
to be used by any test.
But essentially if you run an end-to-end test using SqlToRelConverter, you
should be able to configure the converter to use an empty convertlet table,
which will leave Between unchanged.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
