pnowojski commented on a change in pull request #7938: [FLINK-10941] Keep slots which contain
unconsumed result partitions (on top of #7186)
URL: https://github.com/apache/flink/pull/7938#discussion_r273037957
##########
File path: flink-runtime/src/main/java/org/apache/flink/runtime/resourcemanager/slotmanager/SlotManager.java
##########
@@ -120,15 +120,20 @@
/** True iff the component has been started. */
private boolean started;
+ /** Release task executor only when each produced result partition is either consumed or
failed.. */
+ private final boolean waitResultConsumedToRelease;
+
public SlotManager(
- ScheduledExecutor scheduledExecutor,
- Time taskManagerRequestTimeout,
- Time slotRequestTimeout,
- Time taskManagerTimeout) {
+ ScheduledExecutor scheduledExecutor,
+ Time taskManagerRequestTimeout,
+ Time slotRequestTimeout,
+ Time taskManagerTimeout,
+ boolean waitResultConsumedToRelease) {
Review comment:
maybe add an overloaded constructor with the default value for `waitResultConsumedToRelease`?
Now it's causing quite some code duplication.
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
With regards,
Apache Git Services
|