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

BookKeeper.createLedger(..) should not mask the error with ZKException

    XMLWordPrintableJSON

Details

    • Bug
    • Status: Resolved
    • Major
    • Resolution: Fixed
    • 4.2.0
    • 4.2.2, 4.3.0
    • bookkeeper-server
    • None

    Description

      in BookKeeper.createLedger() following code is masking the error with ZKException. Should throw the original exception to client.

              if (counter.getLh() == null) {
                  LOG.error("ZooKeeper error: " + counter.getrc());
                  throw BKException.create(Code.ZKException);
              }

      Attachments

        1. 0001-BOOKKEEPER-446-create-ledger-exceptions.patch
          8 kB
          Rakesh Radhakrishnan
        2. BOOKKEEPER-446.patch
          14 kB
          Sijie Guo
        3. BOOKKEEPER-446.patch
          14 kB
          Sijie Guo

        Activity

          People

            hustlmsp Sijie Guo
            vinayakumarb Vinayakumar B
            Votes:
            0 Vote for this issue
            Watchers:
            6 Start watching this issue

            Dates

              Created:
              Updated:
              Resolved: