Uploaded image for project: 'ZooKeeper'
  1. ZooKeeper
  2. ZOOKEEPER-1350

Please make JMX registration optional in LearnerZooKeeperServer

    Details

    • Type: Improvement
    • Status: Patch Available
    • Priority: Major
    • Resolution: Unresolved
    • Affects Version/s: 3.4.0
    • Fix Version/s: 3.6.0, 3.5.5
    • Component/s: server
    • Labels:
      None

      Description

      LearnerZooKeeperServer has no option to disable JMX registrations. Curator has a test ZK server cluster. Due to the intricacies of JMX, the registrations cannot be easily undone. In order for the Curator Test cluster to be re-usable in a testing session, JavaAssist ugliness was necessary to make LearnerZooKeeperServer.registerJMX() and LearnerZooKeeperServer.unregisterJMX() NOPs.

      I suggest a simple System property.

        Attachments

        1. jmx_optional.diff
          4 kB
          Jordan Zimmerman
        2. patch.txt
          19 kB
          Jordan Zimmerman
        3. ZOOKEEPER-1350.diff
          40 kB
          Stevo Slavic
        4. ZOOKEEPER-1350-trunk.diff
          40 kB
          Stevo Slavic

          Activity

            People

            • Assignee:
              randgalt Jordan Zimmerman
              Reporter:
              randgalt Jordan Zimmerman
            • Votes:
              4 Vote for this issue
              Watchers:
              5 Start watching this issue

              Dates

              • Created:
                Updated: