Details

    • Type: Sub-task Sub-task
    • Status: Resolved
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: Auto failover (HDFS-3042)
    • Fix Version/s: Auto failover (HDFS-3042)
    • Component/s: auto-failover
    • Labels:
      None

      Description

      Currently the ZKFC is configured with a global set of configs. But different nameservices may want to use different ZooKeeper quorums, ACLs, znodes, etc. So, we should optionally allow the confs to be specified with a nameservice suffix, overriding the same config with no suffix.

      1. hdfs-3200.txt
        4 kB
        Todd Lipcon
      2. hdfs-3200.txt
        4 kB
        Todd Lipcon

        Activity

        Hide
        Todd Lipcon added a comment -

        Attached patch is against the branch, and implements the above. The ZKFC configs can all be suffixed with nameservice or namenode

        Show
        Todd Lipcon added a comment - Attached patch is against the branch, and implements the above. The ZKFC configs can all be suffixed with nameservice or namenode
        Hide
        Aaron T. Myers added a comment -

        Patch looks pretty good to me, Todd. One small comment. +1 once this is addressed:

        1. s/namespace/nameservice/g in "to be overridden on a per-namespace or per-namenode basis."
        Show
        Aaron T. Myers added a comment - Patch looks pretty good to me, Todd. One small comment. +1 once this is addressed: s/namespace/nameservice/g in "to be overridden on a per-namespace or per-namenode basis."
        Hide
        Todd Lipcon added a comment -

        Attached patch fixes the typo. Thanks for catching my slipup. I'll commit this to the auto-HA branch momentarily.

        Show
        Todd Lipcon added a comment - Attached patch fixes the typo. Thanks for catching my slipup. I'll commit this to the auto-HA branch momentarily.

          People

          • Assignee:
            Todd Lipcon
            Reporter:
            Todd Lipcon
          • Votes:
            0 Vote for this issue
            Watchers:
            3 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development