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

ZooKeeper can revert to old data when a snapshot is created outside of normal processing

    XMLWordPrintableJSON

Details

    • Bug
    • Status: Closed
    • Blocker
    • Resolution: Fixed
    • 3.1.1, 3.2.1
    • 3.1.2, 3.2.2, 3.3.0
    • server
    • None
    • Reviewed
    • fixed bug in zookeeper that can lead zookeeper to revert to old data when a snapshot is created without a corresponding log.

    Description

      when zookeeper starts up it will restore the most recent state (latest zxid) it finds in the data directory. unfortunately, in the quorum version of zookeeper updates are logged using an epoch based on the latest log file in a directory. if there is a snapshot with a higher epoch than the log files, the zookeeper server will start logging using an epoch one higher than the highest log file.

      so if a data directory has a snapshot with an epoch of 27 and there are no log files, zookeeper will start logging changes using epoch 1. if the cluster restarts the state will be restored from the snapshot with the epoch of 27, which in effect, restores old data.

      normal operation of zookeeper will never result in this situation.

      this does not effect standalone zookeeper.

      a fix should make sure to use an epoch one higher than the current state, whether it comes from the snapshot or log, and should include a sanity check to make sure that a follower never connects to a leader that has a lower epoch than its own.

      Attachments

        1. test.patch
          13 kB
          Benjamin Reed
        2. ZOOKEEPER-582_3.1.patch
          13 kB
          Mahadev Konar
        3. ZOOKEEPER-582_3.2.patch
          14 kB
          Mahadev Konar
        4. ZOOKEEPER-582_3.2.patch
          3 kB
          Mahadev Konar
        5. ZOOKEEPER-582.patch
          14 kB
          Mahadev Konar
        6. ZOOKEEPER-582.patch
          14 kB
          Mahadev Konar
        7. ZOOKEEPER-582.patch
          14 kB
          Mahadev Konar
        8. ZOOKEEPER-582.patch
          14 kB
          Mahadev Konar
        9. ZOOKEEPER-582.patch
          3 kB
          Mahadev Konar

        Activity

          People

            mahadev Mahadev Konar
            breed Benjamin Reed
            Votes:
            0 Vote for this issue
            Watchers:
            1 Start watching this issue

            Dates

              Created:
              Updated:
              Resolved: