jacques-n edited a comment on pull request #2531: URL: https://github.com/apache/calcite/pull/2531#issuecomment-925165013
Just a small correction @julianhyde > Removing the `with` methods is a massive negative because of the disruption to dependent projects. That's incorrect interpretation of the proposed changes. The `with` methods remain without any disruption. Only the `RelRule.Config.as` method and the `RelRule.Config.EMPTY` singleton would be removed. The latter is removed because it doesn't make sense without the `as` method. -- 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]
