xiedeyantu commented on PR #5354:
URL: https://github.com/apache/calcite/pull/5354#issuecomment-6393129162

   > We either make it clear that set_expand(true) is deprecated, or we need to 
fix bugs on that code path.
   
   I agree with Mihai's idea that gradually switching to expand=false is a very 
good direction. I remember that in the early days, expand=true appeared because 
there were no well-established decorrelation rules. I seem to recall that 
Julian also mentioned this in some JIRA. I'm inclined to think that we can 
gradually deprecate expand=true.


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

Reply via email to