LEEKYE commented on code in PR #35120: URL: https://github.com/apache/beam/pull/35120#discussion_r2193266723
########## sdks/java/harness/src/main/java/org/apache/beam/fn/harness/control/ExecutionStateSampler.java: ########## @@ -65,6 +68,7 @@ public class ExecutionStateSampler { private static final Logger LOG = LoggerFactory.getLogger(ExecutionStateSampler.class); private static final int DEFAULT_SAMPLING_PERIOD_MS = 200; private static final long MAX_LULL_TIME_MS = TimeUnit.MINUTES.toMillis(5); + private static final long MIN_LULL_TIME_MS_FOR_RESTART = TimeUnit.MINUTES.toMillis(10); Review Comment: DONE. I removed the lower bound for timeout -- 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: github-unsubscr...@beam.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org