mihaibudiu commented on PR #4620: URL: https://github.com/apache/calcite/pull/4620#issuecomment-3529141069
I think that the visitor should have methods for all Rel classes that are in core. It is a breaking change, but I think it's a bug - the methods should have been added when the LogicalSnapshot was added. But that's just my opinion, our compiler moves very quickly to accommodate such changes. -- 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]
