Jackie-Jiang commented on PR #10199: URL: https://github.com/apache/pinot/pull/10199#issuecomment-1407796478
> hey @Jackie-Jiang and @jackjlli looks like we use this class in one of our LinkedIn MPs. Can we deprecate this instead of removing this outright just to give us a chance to remove the code that uses this properly? I don't have full context on why we are using this yet but am looking into it > > Removal of this class is causing our builds to fail and will become a release blocker Marking it as deprecated won't solve the problem of management overhead. Can you check if you can simply remove it from the MP? I feel that should be some dead code that should be cleaned up. If that is indeed in use, we can add this class back. -- 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] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
