asolimando commented on pull request #2688: URL: https://github.com/apache/calcite/pull/2688#issuecomment-1011168103
CI/CD won't necessarily break if you remove them, they might be unused _within_ Calcite, but they can be very well used by Calcite users, removing them might break their code. What others suggested, going through a `@Deprecated` annotation is what we generally do. However, it would be good to at least comment on why some methods are deprecated, and what to use instead in order to achieve the same goal, if possible (e.g., multiple versions of a constructor for a given class). -- 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]
