[GitHub] flink issue #5461: [FLINK-8475][config][docs] Integrate Mesos options

2018-02-12 Thread zentol
Github user zentol commented on the issue:

https://github.com/apache/flink/pull/5461
  
yes, it will show up in the HA table. merging.


---


[GitHub] flink issue #5461: [FLINK-8475][config][docs] Integrate Mesos options

2018-02-12 Thread aljoscha
Github user aljoscha commented on the issue:

https://github.com/apache/flink/pull/5461
  
So when we get to the HA options this will show up again? If yes, please go 
ahead.


---


[GitHub] flink issue #5461: [FLINK-8475][config][docs] Integrate Mesos options

2018-02-12 Thread zentol
Github user zentol commented on the issue:

https://github.com/apache/flink/pull/5461
  
Currently we assign each `ConfigOption` to exactly one table, so there's 
little we can do at the moment about this option.


---


[GitHub] flink issue #5461: [FLINK-8475][config][docs] Integrate Mesos options

2018-02-12 Thread aljoscha
Github user aljoscha commented on the issue:

https://github.com/apache/flink/pull/5461
  
And I like how the options where not sorted before and also related options 
where not next to each other ... 😅 


---


[GitHub] flink issue #5461: [FLINK-8475][config][docs] Integrate Mesos options

2018-02-12 Thread aljoscha
Github user aljoscha commented on the issue:

https://github.com/apache/flink/pull/5461
  
Looks good except for the one question I had! 👍 


---


[GitHub] flink issue #5461: [FLINK-8475][config][docs] Integrate Mesos options

2018-02-12 Thread aljoscha
Github user aljoscha commented on the issue:

https://github.com/apache/flink/pull/5461
  
The current documentation has 
`high-availability.zookeeper.path.mesos-workers: The ZooKeeper root path for 
persisting the Mesos worker information.` Do you know why this is not there 
anymore?


---