ZooKeeper
  1. ZooKeeper
  2. ZOOKEEPER-862

Hedwig created ledgers with hardcoded Bookkeeper ensemble and quorum size. Make these a server config parameter instead.

    Details

    • Type: Improvement Improvement
    • Status: Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: 3.4.0
    • Component/s: contrib-hedwig
    • Labels:
      None
    • Hadoop Flags:
      Reviewed

      Description

      Hedwig code right now when using Bookkeeper as the persistence store is hardcoding the number of bookie servers in the ensemble and quorum size. This is used the first time a ledger is created. This should be exposed as a server configuration parameter instead.

        Activity

        Mahadev konar made changes -
        Status Resolved [ 5 ] Closed [ 6 ]
        Benjamin Reed made changes -
        Status Patch Available [ 10002 ] Resolved [ 5 ]
        Hadoop Flags [Reviewed]
        Resolution Fixed [ 1 ]
        Hide
        Benjamin Reed added a comment -

        +1 looks good thanx Erwin

        it looks like this was accidentally committed in r1031051

        Show
        Benjamin Reed added a comment - +1 looks good thanx Erwin it looks like this was accidentally committed in r1031051
        Erwin Tam made changes -
        Status Open [ 1 ] Patch Available [ 10002 ]
        Hide
        Erwin Tam added a comment -

        Fix so the bookkeeper ledgers created will use a server configuration parameter to determine the ensemble and quorum size instead of hardcoding it.

        Show
        Erwin Tam added a comment - Fix so the bookkeeper ledgers created will use a server configuration parameter to determine the ensemble and quorum size instead of hardcoding it.
        Erwin Tam made changes -
        Field Original Value New Value
        Attachment ZOOKEEPER-862.patch [ 12453709 ]
        Erwin Tam created issue -

          People

          • Assignee:
            Erwin Tam
            Reporter:
            Erwin Tam
          • Votes:
            0 Vote for this issue
            Watchers:
            1 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development