FatLittle commented on a change in pull request #1991:
URL: https://github.com/apache/calcite/pull/1991#discussion_r436254672
##########
File path: core/src/test/java/org/apache/calcite/test/JdbcAdapterTest.java
##########
@@ -179,6 +180,7 @@
@Test void testPushDownSort() {
CalciteAssert.model(JdbcTest.SCOTT_MODEL)
+ .with(CalciteConnectionProperty.TOPDOWN_OPT.camelName(), false)
Review comment:
In this case, the new planner is producing a plan with exactly the same
cost.
However, the new plan will affect later tests' assertion. I didn't want to
modified so many of them. So I added this line instead of modifying the
expects.
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
[email protected]
