ZooKeeper
  1. ZooKeeper
  2. ZOOKEEPER-1350

Please make JMX registration optional in LearnerZooKeeperServer

    Details

    • Type: Improvement Improvement
    • Status: Open
    • Priority: Major Major
    • Resolution: Unresolved
    • Affects Version/s: 3.4.0
    • Fix Version/s: 3.5.0
    • 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.

      1. patch.txt
        19 kB
        Jordan Zimmerman
      2. jmx_optional.diff
        4 kB
        Jordan Zimmerman

        Activity

        Jordan Zimmerman created issue -
        Jordan Zimmerman made changes -
        Field Original Value New Value
        Attachment jmx_optional.diff [ 12509347 ]
        Jordan Zimmerman made changes -
        Status Open [ 1 ] Patch Available [ 10002 ]
        Patrick Hunt made changes -
        Assignee Jordan Zimmerman [ randgalt ]
        Patrick Hunt made changes -
        Status Patch Available [ 10002 ] Open [ 1 ]
        Jordan Zimmerman made changes -
        Attachment patch.txt [ 12509655 ]
        Jordan Zimmerman made changes -
        Status Open [ 1 ] Patch Available [ 10002 ]
        Mahadev konar made changes -
        Fix Version/s 3.5.0 [ 12316644 ]
        Michi Mutsuzaki made changes -
        Status Patch Available [ 10002 ] Open [ 1 ]

          People

          • Assignee:
            Jordan Zimmerman
            Reporter:
            Jordan Zimmerman
          • Votes:
            1 Vote for this issue
            Watchers:
            2 Start watching this issue

            Dates

            • Created:
              Updated:

              Development