mihaibudiu commented on PR #5031: URL: https://github.com/apache/calcite/pull/5031#issuecomment-4756572646
There so many config flags in Calcite... Thinking of this, having a way to do it in SqlToRelConverter is probably very useful, so we should offer this possibility. Deprecating Uncollect is a big move, it may cause problems for other people who depend on it. We should probably give it some time. SelectMany is actually not a very good name, the "real" selectMany is actually much more general, it takes an arbitrary function which returns an iterator. We could call it LogicalUnnest. Let's see if there are other comments which I can address in an updated commit. I will try to add it to SqlToRel, but not sure I will get to it during the week-end. Happy to get other naming suggestions as well. -- 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]
