[ZOOKEEPER-2642] ZooKeeper reconfig API backward compatibility fix

• Type: Bug
• Status: Closed
• Priority: Blocker
• Resolution: Fixed
• Affects Version/s: 3.5.2
• Fix Version/s: 3.5.3, 3.6.0
• Component/s: c client, java client
• Labels: pull-request-available
• Assignee: Jordan Zimmerman
• Reporter: Jordan Zimmerman
• Created: 07/Dec/16 11:23
• Updated: 30/Jan/19 16:16
• Resolved: 11/Feb/17 23:34

Description

ZOOKEEPER-2014 moved the reconfig() methods into a new class, ZooKeeperAdmin. It appears this was done to document that these are methods have access restrictions. However, this change breaks Apache Curator (and possibly other clients). Curator APIs will have to be changed and/or special methods need to be added. A breaking change of this kind should only be done when the benefit is overwhelming. In this case, the same information can be conveyed with documentation and possibly a deprecation notice.

Revert the creation of the ZooKeeperAdmin class and move the reconfig() methods back to the ZooKeeper class with additional documentation.

Issue Links

• breaks: ZOOKEEPER-2014 Only admin should be allowed to reconfig a cluster
• links to: GitHub Pull Request #122
• links to: GitHub Pull Request #151
• links to: GitHub Pull Request #151
• links to: GitHub Pull Request #152