aho135 commented on PR #20314: URL: https://github.com/apache/druid/pull/20314#issuecomment-5781373712
@FrankChen021 I reverted propagation of the Cancel flag to the HepPlanner that was suggested [here](https://github.com/apache/druid/pull/20314#discussion_r3979110367). The code there didn't seem very clean since there was quite of bit of duplication with Calcite code. In my own testing with large IN clauses the planning time was dominated by the VolcanoPlanner. Curious if you've seen planning bottlenecks in the past in the HepPlanner that would justify the additional code. -- 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] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
