suneet-s commented on code in PR #14752:
URL: https://github.com/apache/druid/pull/14752#discussion_r1286096469


##########
indexing-service/src/main/java/org/apache/druid/indexing/common/task/CompactionTask.java:
##########
@@ -904,6 +928,8 @@ static class ExistingSegmentAnalyzer
     // For processMetricsSpec:
     private final Set<List<AggregatorFactory>> aggregatorFactoryLists = new 
HashSet<>();
 
+    private long processingTimeMillis = 0;

Review Comment:
   removed. I had thought perhaps a metric for all the time spent in this class 
would be interesting, but decided against implementing it as it would have 
resulted in a few new metrics.



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

Reply via email to