StephanEwen commented on issue #9553: [FLINK-13883] Remove unused AkkaOptions related to Akka's death watch URL: https://github.com/apache/flink/pull/9553#issuecomment-526110402 +1 to the change Concerning the point to keep the ConfigOptions for the death watch: I see the point, and it seems to be in line with the handling of the old `ConfigConstants` class. In this case, I would vote to go for cleanup here, to not accumulate too much dead code. The keeping of previous options was one of the reasons that got us into the weird state in the first place (referring to unused death watch options for restart strategy defaults). We still do not break existing config files, we only break programs that refer to the config option fields.
---------------------------------------------------------------- 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: [email protected] With regards, Apache Git Services
