[GitHub] [flink] zhuzhurk edited a comment on issue #9688: [FLINK-13056] [runtime] Introduce FastRestartPipelinedRegionStrategy which has better failover handling performance at the cost of slower region building and more memory used

[GitBox <gi...@apache.org>]

zhuzhurk edited a comment on issue #9688: [FLINK-13056] [runtime] Introduce FastRestartPipelinedRegionStrategy which has better failover handling performance at the cost of slower region building and more memory used
URL: https://github.com/apache/flink/pull/9688#issuecomment-532262379
 
 
   > How do you configure the `FastRestartPipelinedRegionStrategy`?
   
   One option is a boolean config "jobmanager.execution.failover-strategy.region.fast-mode". This reveals to users that we have a fast mode for region failover.
   Another option is a new supported value "region-fast" for the failover strategy specification config ("jobmanager.execution.failover-strategy"). This reveals to users a new region failover strategy.
   I prefer the first option which hides the implementation and makes it easy if later we'd like to keep only one region failover strategy. 
   What's your opinion?
   
   I planned to add the config when using it for the legacy failover strategy creation. But as you mentioned we can also add it in this PR.

----------------------------------------------------------------
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