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

          Flavio Junqueira created issue -
          Flavio Junqueira made changes -
          Field Original Value New Value
          Link This issue blocks ZOOKEEPER-503 [ ZOOKEEPER-503 ]
          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.
          Flavio Junqueira made changes -
          Attachment ZOOKEEPER-461.patch [ 12416124 ]
          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.
          Flavio Junqueira made changes -
          Attachment ZOOKEEPER-461.patch [ 12416208 ]
          Benjamin Reed made changes -
          Link This issue is part of ZOOKEEPER-503 [ ZOOKEEPER-503 ]
          Flavio Junqueira made changes -
          Link This issue is part of ZOOKEEPER-507 [ ZOOKEEPER-507 ]
          Mahadev konar made changes -
          Assignee Flavio Paiva Junqueira [ fpj ]
          Hide
          Mahadev konar added a comment -

          Committed in ZOOKEEPER-507.

          Show
          Mahadev konar added a comment - Committed in ZOOKEEPER-507 .
          Mahadev konar made changes -
          Status Open [ 1 ] Resolved [ 5 ]
          Fix Version/s 3.3.0 [ 12313976 ]
          Resolution Fixed [ 1 ]
          Patrick Hunt made changes -
          Status Resolved [ 5 ] Closed [ 6 ]
          Gavin made changes -
          Link This issue blocks ZOOKEEPER-503 [ ZOOKEEPER-503 ]
          Gavin made changes -
          Link This issue is depended upon by ZOOKEEPER-503 [ ZOOKEEPER-503 ]
          Transition Time In Source Status Execution Times Last Executer Last Execution Date
          Open Open Resolved Resolved
          199d 15h 43m 1 Mahadev konar 26/Jan/10 23:19
          Resolved Resolved Closed Closed
          58d 18h 5m 1 Patrick Hunt 26/Mar/10 17:24

            People

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

              Dates

              • Created:
                Updated:
                Resolved:

                Development