lukecwik commented on code in PR #25291:
URL: https://github.com/apache/beam/pull/25291#discussion_r1096382075
##########
sdks/java/harness/src/main/java/org/apache/beam/fn/harness/control/ExecutionStateSampler.java:
##########
@@ -187,13 +253,37 @@ public class ExecutionStateTracker implements
BundleProgressReporter {
// Read and written by the ExecutionStateSampler thread
private long transitionsAtLastSample;
+ // Ignore the @UnderInitialization for metricsContainer since both this
and it will be
Review Comment:
It still will bring this up since the thing under initialization is the
execution state tracker. Fixed up comment.
--
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]
