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

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

    XMLWordPrintableJSON

    Details

    • Type: Bug
    • Status: Resolved
    • Priority: Major
    • Resolution: Fixed
    • Affects Version/s: 4.2.0
    • Fix Version/s: 4.2.2, 4.3.0
    • Component/s: bookkeeper-server
    • Labels:
      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. BOOKKEEPER-446.patch
          14 kB
          Sijie Guo
        2. BOOKKEEPER-446.patch
          14 kB
          Sijie Guo
        3. 0001-BOOKKEEPER-446-create-ledger-exceptions.patch
          8 kB
          Rakesh Radhakrishnan

          Activity

            People

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

              Dates

              • Created:
                Updated:
                Resolved: