Uploaded image for project: 'Bookkeeper'
  1. Bookkeeper
  2. BOOKKEEPER-112

Bookie Recovery on an open ledger will cause LedgerHandle#close on that ledger to fail

    Details

    • Type: Bug
    • Status: Closed
    • Priority: Major
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: 4.1.0
    • Component/s: None
    • Labels:
      None

      Description

      Bookie recovery updates the ledger metadata in zookeeper. LedgerHandle will not get notified of this update, so it will try to write out its own ledger metadata, only to fail with KeeperException.BadVersion. This effectively fences all write operations on the LedgerHandle (close and addEntry). close will fail for obvious reasons. addEntry will fail once it gets to the failed bookie in the schedule, tries to write, fails, selects a new bookie and tries to update ledger metadata.

      Update Line 605, testSyncBookieRecoveryToRandomBookiesCheckForDupes(), when done
      Also, uncomment addEntry in TestFencing#testFencingInteractionWithBookieRecovery()

        Attachments

        1. BK-112.patch
          18 kB
          Sijie Guo
        2. bk-112.pdf
          43 kB
          Flavio Junqueira
        3. bk-112.pdf
          37 kB
          Flavio Junqueira
        4. BOOKKEEPER-112.patch
          54 kB
          Sijie Guo
        5. BOOKKEEPER-112.patch_v2
          31 kB
          Sijie Guo
        6. BOOKKEEPER-112.patch_v3
          33 kB
          Sijie Guo
        7. BOOKKEEPER-112.patch_v4
          44 kB
          Sijie Guo
        8. BOOKKEEPER-112.patch_v5
          45 kB
          Sijie Guo
        9. BOOKKEEPER-112.patch_v6
          39 kB
          Sijie Guo
        10. BOOKKEEPER-112.patch_v7
          39 kB
          Sijie Guo
        11. BOOKKEEPER-112.patch_v8
          39 kB
          Sijie Guo

          Issue Links

            Activity

              People

              • Assignee:
                hustlmsp Sijie Guo
                Reporter:
                fpj Flavio Junqueira
              • Votes:
                0 Vote for this issue
                Watchers:
                2 Start watching this issue

                Dates

                • Created:
                  Updated:
                  Resolved: