ZooKeeper
  1. ZooKeeper
  2. ZOOKEEPER-461

Condense ledger configuration in ZooKeeper

    Details

    • Type: Improvement Improvement
    • Status: Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: 3.3.0
    • Component/s: contrib-bookkeeper
    • Labels:
      None

      Description

      We currently use several znodes to represent the configuration and state of a ledger in ZooKeeper. Although this is good for readability, it makes operations such as create, close, and open more complex, in particular the asynchronous versions of these operations. The main idea of this jira is to store as much as possible of the configuration of a ledger as data of a single znode.

      1. ZOOKEEPER-461.patch
        101 kB
        Flavio Junqueira
      2. ZOOKEEPER-461.patch
        97 kB
        Flavio Junqueira

        Issue Links

          Activity

          Hide
          Flavio Junqueira added a comment -

          Attaching preliminary patch. It is not passing unit tests yes.

          Show
          Flavio Junqueira added a comment - Attaching preliminary patch. It is not passing unit tests yes.
          Hide
          Flavio Junqueira added a comment -

          Updated patch. Only two tests are failing now: BookieFailureTest and LedgerRecoveryTest. All others pass for me, and AsyncLedgerOpsTest seems to work fine. I have updated it to fix problems pointed out in another jira.

          Show
          Flavio Junqueira added a comment - Updated patch. Only two tests are failing now: BookieFailureTest and LedgerRecoveryTest. All others pass for me, and AsyncLedgerOpsTest seems to work fine. I have updated it to fix problems pointed out in another jira.
          Hide
          Mahadev konar added a comment -

          Committed in ZOOKEEPER-507.

          Show
          Mahadev konar added a comment - Committed in ZOOKEEPER-507 .

            People

            • Assignee:
              Flavio Junqueira
              Reporter:
              Flavio Junqueira
            • Votes:
              0 Vote for this issue
              Watchers:
              1 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved:

                Development