sjvanrossum commented on PR #33247: URL: https://github.com/apache/beam/pull/33247#issuecomment-2506245359
Converting to draft, misread the implementation of `FinalizeBundleHandler` which only retains a single thread to perform its tasks. This change would have every finalize task co-schedule onto the execution thread pool. Depending on the perf impact of scheduling on the same pool, this PR can be amended to have `FinalizeBundleHandler` use a dedicated single threaded `ScheduledExecutorService` which should be equivalent to the current implementation that only differs in the amount of lines to maintain. -- 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]
