smengcl commented on pull request #2645: URL: https://github.com/apache/hadoop/pull/2645#issuecomment-768251870
> @smengcl @iwasakims Thank you for the work here > I have taken your latest patch, compiled and trying to verify trace feature. > Just wanted to confirm, As per the PR, TraceAdminProtocols have been removed. Will these TraceAdminProtocol be handled in any follow-up JIRAs or will be removed permanently ? Hi @Sushmasree-28 , thanks for taking your time to check this PR. The current plan it to ditch the `TraceAdminProtocol` **completely**. As I previously (tentatively) implemented OpenTracing in https://github.com/apache/hadoop/pull/1846 , I don't find the legacy interface useful anymore (unless we want to maintain compatibility with HTrace -- which has become a potential security hazard -- and hence this PR). So I would just remove it here. Let me know if you have more concerns about it. ---------------------------------------------------------------- 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] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
