liyafan82 commented on pull request #1944: URL: https://github.com/apache/calcite/pull/1944#issuecomment-652971498
> LGTM, but any issue with changing the interface with default logic ? Thanks a lot for you feedback. We want to prevent the client from overriding the default comparison logic (isLt, isLe, and equals). We cannot do this with an interface. With an abstract class, however, this can be easily achieved by declaring the method as final. ---------------------------------------------------------------- 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]
