mihaibudiu commented on PR #5354: URL: https://github.com/apache/calcite/pull/5354#issuecomment-6387188859
> I’m wondering whether we still need to maintain the expand=true mode. We should try to address these issues in expand=false mode wherever possible. The issue exposed by the last SQL query in the comments on [CALCITE-4340](https://issues.apache.org/jira/browse/CALCITE-4340) is actually what [CALCITE-7278](https://issues.apache.org/jira/browse/CALCITE-7278) aims to resolve. This is a question for the dev list. Our compiler is still using expand=true but I have actually run all our tests successfully with expand=false, so we will try to switch. But there may be many other projects that rely on expand=true. We could try to deprecate the setExpand method to see who complains. -- 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]
