ctubbsii commented on PR #5398: URL: https://github.com/apache/accumulo/pull/5398#issuecomment-3300225117
I think this can be closed in favor of #5691. I don't think the additional tracing changes are worth pursuing anymore. The premise for those changes is that the protocol could create its own trace Span object that spans the entirety of the RPC operation, and could be used as a parent Span for what happens on the server side. However, I don't think that's the case anymore... or at least, it's not easy to do... since the `writeMessageEnd` can finish before the operation on the server side, and there may not be a response from the server to close that Span. -- 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]
