vrajat commented on code in PR #16004:
URL: https://github.com/apache/pinot/pull/16004#discussion_r2136881578
##########
pinot-spi/src/main/java/org/apache/pinot/spi/trace/Tracing.java:
##########
@@ -172,22 +171,23 @@ public InvocationRecording activeRecording() {
*/
public static class DefaultThreadResourceUsageAccountant implements
ThreadResourceUsageAccountant {
Review Comment:
That will be a major change and complicate this PR. However I predict that
in the course of working on #16039 , this will disappear. The main complication
is what to return when `Tracing.getThreadAccountant` is called.
--
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: commits-unsubscr...@pinot.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscr...@pinot.apache.org
For additional commands, e-mail: commits-h...@pinot.apache.org
