Github user mxm commented on a diff in the pull request:

    https://github.com/apache/flink/pull/2249#discussion_r71311352
  
    --- Diff: docs/setup/config.md ---
    @@ -272,7 +272,9 @@ For example when running Flink on YARN on an 
environment with a restrictive fire
     
     - `recovery.zookeeper.quorum`: Defines the ZooKeeper quorum URL which is 
used to connet to the ZooKeeper cluster when the 'zookeeper' recovery mode is 
selected
     
    -- `recovery.zookeeper.path.root`: (Default '/flink') Defines the root dir 
under which the ZooKeeper recovery mode will create znodes.
    +- `recovery.zookeeper.path.root`: (Default '/flink') Defines the root dir 
under which the ZooKeeper recovery mode will create namespace directories.
    +
    +- `recovery.zookeeper.path.namespace`: (Default '/default_ns' in 
standalone mode, or the <yarn-application-id> under Yarn) Defines the 
subdirectory under the root dir where the ZooKeeper recovery mode will create 
znodes. This allows to isolate multiple applications on the same ZooKeeper. 
    --- End diff --
    
    You're right, it would just be a heuristic for generating a unique 
namespace but it is not unique because multiple Flink instances might be 
running on the same host names (e.g. multiple processes, Docker containers). So 
+1 for not adding something that can cause hard to debug problems. We have 
covered the Yarn use case which makes it very convenient to setup a namespace.


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

Reply via email to