Github user tillrohrmann commented on the issue:
https://github.com/apache/flink/pull/2905
I fear that we cannot merge the PR as it is, because it would leave the
`CuratorFramework` client open for some of the components used in the master.
For example, the `WebMonitor` uses the same
Github user StephanEwen commented on the issue:
https://github.com/apache/flink/pull/2905
Ah, this refers only to `flip-6` - then it makes sense :-)
Will merge this...
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as
Github user shuai-xu commented on the issue:
https://github.com/apache/flink/pull/2905
@StephanEwen yes, the client are both started and stopped in
ZookeeperHaServices.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If
Github user StephanEwen commented on the issue:
https://github.com/apache/flink/pull/2905
Thanks for the patch!
Is there another place where the client now gets shut down properly?
---
If your project is set up for it, you can reply to this email and have your
reply appear on