yunfengzhou-hub commented on code in PR #24667: URL: https://github.com/apache/flink/pull/24667#discussion_r1575609479
########## flink-runtime/src/main/java/org/apache/flink/runtime/asyncprocessing/AsyncExecutionController.java: ########## @@ -94,29 +108,91 @@ public class AsyncExecutionController<K> { */ final AtomicInteger inFlightRecordNum; + /** + * The flag to indicate whether the {@link #bufferTimeOut} is reached, if yes, a trigger will + * perform actively when the next state request arrives even if the activeQueue has not reached + * the {@link #batchSize}. + */ + final AtomicBoolean timeoutFlag; + + /** The executor service that schedules and calls the triggers of this task. */ + final ScheduledThreadPoolExecutor scheduledExecutor; Review Comment: Would it be better to reuse existing utility methods like `FutureUtils.delay()`? This way AEC won't need to maintain such resources. -- 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: issues-unsubscr...@flink.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org