xuzifu666 edited a comment on pull request #2688:
URL: https://github.com/apache/calcite/pull/2688#issuecomment-1011117476


   > 
   i found no place use these method in whole project or tests,so i though 
these methods are useless. if these methods are used at a place in Calcite or 
tests, the ci/cd would not be successful, but we can see ci/cd success. i added 
  @Deprecated on these methods now. whether remove can decide in next version. 
now i though mark it deprecated is nesscessary. @rubenada  do you aggree with 
me? 
   


-- 
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]


Reply via email to