iwanttobepowerful commented on PR #5031: URL: https://github.com/apache/calcite/pull/5031#issuecomment-4756310376
Generating LogicalSelectManydirectly within SqlToRelConverterwould feel cleaner and more natural, I think. I’m fully on board with deprecating Uncollectin favor of this new operator. Also, would it be reasonable to rename LogicalSelectManyto LogicalUnnest? The current name feels a bit too LINQ-specific for a core Calcite relational operator. -- 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]
