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

ZOOKEEPER-2355 broke Curator TestingQuorumPeerMain

Log workAgile BoardRank to TopRank to BottomAttach filesAttach ScreenshotBulk Copy AttachmentsBulk Move AttachmentsVotersWatch issueWatchersCreate sub-taskConvert to sub-taskMoveLinkCloneLabelsUpdate Comment AuthorReplace String in CommentUpdate Comment VisibilityDelete Comments
    XMLWordPrintableJSON

Details

    • Bug
    • Status: Resolved
    • Major
    • Resolution: Not A Problem
    • 3.5.3, 3.4.11
    • None
    • None

    Description

      ZOOKEEPER-2355 added a getQuorumPeer method to QuorumPeerMain https://github.com/apache/zookeeper/blob/release-3.5.3/src/java/main/org/apache/zookeeper/server/quorum/QuorumPeerMain.java#L194. TestingQuorumPeerMain has an identically named method, which is now unintentionally overridding the one in the base class.

      This is fixed by CURATOR-409, however, I'd like this to be fixed in ZooKeeper as well.

      Attachments

        Issue Links

        is related to

        Sub-task - The sub-task of the issue KNOX-1599 Upgrade zookeeper to 3.4.13 and curator-test to 2.13.0

        • Minor - Minor loss of function, or other problem where easy workaround is present.
        • Closed

        Bug - A problem which impairs or prevents the functions of the product. HADOOP-15974 Upgrade Curator version to 2.13.0 to fix ZK tests

        • Major - Major loss of function.
        • Resolved
        relates to

        Bug - A problem which impairs or prevents the functions of the product. CURATOR-409 TestingQuorumPeerMain does not work with Zookeeper 3.5.3-beta

        • Major - Major loss of function.
        • Resolved
        links to

        Web Link GitHub Pull Request #676

        Activity
          This comment will be Viewable by All Users Viewable by All Users
          Cancel

          People

            Unassigned Unassigned Assign to me
            aajisaka Akira Ajisaka
            Votes:
            0 Vote for this issue
            Watchers:
            3 Start watching this issue

            Dates

              Created:
              Updated:
              Resolved:

              Time Tracking

              Estimated:
              Original Estimate - Not Specified
              Not Specified
              Remaining:
              Remaining Estimate - 0h
              0h
              Logged:
              Time Spent - 3h 10m
              3h 10m

              Slack

                Issue deployment