FatLittle commented on a change in pull request #1991:
URL: https://github.com/apache/calcite/pull/1991#discussion_r432252714



##########
File path: 
kafka/src/test/java/org/apache/calcite/adapter/kafka/KafkaAdapterTest.java
##########
@@ -73,6 +74,7 @@
 
   @Test void testFilterWithProject() {
     assertModel(MODEL)
+        .with(CalciteConnectionProperty.TOP_DOWN_OPT.camelName(), false)

Review comment:
       skip using top-down optimization here. This case will produce another 
plan with the same cost as the expect plan 
    (Two candidates are both in the memo but the new best plan is produced 
first) but it will hang when executing the new plan. Still don't know why it 
hangs. 




----------------------------------------------------------------
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]


Reply via email to