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

Last processed zxid set prematurely while establishing leadership

VotersWatch issueWatchersCreate sub-taskLinkCloneUpdate Comment AuthorReplace String in CommentUpdate Comment VisibilityDelete Comments
    XMLWordPrintableJSON

Details

    • Bug
    • Status: Closed
    • Blocker
    • Resolution: Fixed
    • 3.3.1
    • 3.3.2, 3.4.0
    • quorum
    • None
    • Reviewed

    Description

      The leader code is setting the last processed zxid to the first of the new epoch even before connecting to a quorum of followers. Because the leader code sets this value before connecting to a quorum of followers (Leader.java:281) and the follower code throws an IOException (Follower.java:73) if the leader epoch is smaller, we have that when the false leader drops leadership and becomes a follower, it finds a smaller epoch and kills itself.

      Attachments

        1. ZOOKEEPER-790.patch
          13 kB
          Flavio Paiva Junqueira
        2. ZOOKEEPER-790.patch
          13 kB
          Flavio Paiva Junqueira
        3. ZOOKEEPER-790.patch
          13 kB
          Flavio Paiva Junqueira
        4. ZOOKEEPER-790.patch
          3 kB
          Flavio Paiva Junqueira
        5. ZOOKEEPER-790.patch
          1 kB
          Flavio Paiva Junqueira
        6. ZOOKEEPER-790.travis.log.bz2
          44 kB
          Travis Crawford
        7. ZOOKEEPER-790.v2.patch
          17 kB
          Flavio Paiva Junqueira
        8. ZOOKEEPER-790.v2.patch
          17 kB
          Flavio Paiva Junqueira
        9. ZOOKEEPER-790.v2.patch
          4 kB
          Flavio Paiva Junqueira
        10. ZOOKEEPER-790-3.3.patch
          13 kB
          Flavio Paiva Junqueira
        11. ZOOKEEPER-790-3.3.patch
          12 kB
          Flavio Paiva Junqueira
        12. ZOOKEEPER-790-3.3.v2.patch
          17 kB
          Flavio Paiva Junqueira
        13. ZOOKEEPER-790-follower-request-NPE.log
          16 kB
          Sergey Doroshenko
        14. ZOOKEEPER-790-test.patch
          10 kB
          Sergey Doroshenko

        Issue Links

        Activity

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

          People

            fpj Flavio Paiva Junqueira
            fpj Flavio Paiva Junqueira
            Votes:
            0 Vote for this issue
            Watchers:
            2 Start watching this issue

            Dates

              Created:
              Updated:
              Resolved:

              Slack

                Issue deployment