xndai commented on pull request #1976: URL: https://github.com/apache/calcite/pull/1976#issuecomment-628798575
If this's something we believe should be enforced, why don't we just add such check? There are a number of rules you can be applied to both logical and physical nodes today (matching the base rel class instead of the logical rel). Those rules won't fire anymore with your change. Certain scenarios could break due to this. ---------------------------------------------------------------- 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. For queries about this service, please contact Infrastructure at: [email protected]
