Bookkeeper
  1. Bookkeeper
  2. BOOKKEEPER-216

Bookie doesn't exit with right exit code

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Minor Minor
    • Resolution: Fixed
    • Affects Version/s: 4.1.0
    • Fix Version/s: 4.1.0
    • Component/s: None
    • Labels:
      None

      Description

      there is typo in Bookie#shutdown. so it doesn't quit with right exit code when encountering exceptions.

          synchronized int shutdown(int exitCode) {
              try {
                  if (running) { // avoid shutdown twice
                      // the exitCode only set when first shutdown usually due to exception found
                      this.exitCode = exitCode;
                      // mark bookie as in shutting down progress
                      shuttingdown = true;
                      // shut down gc thread, which depends on zookeeper client
                      // also compaction will write entries again to entry log file
                      gcThread.shutdown();
                      // Shutdown the ZK client
                      if(zk != null) zk.close();
                      this.interrupt();
                      this.join();
                      syncThread.shutdown();
      
                      // Shutdown the EntryLogger which has the GarbageCollector Thread running
                      entryLogger.shutdown();
                      // close Ledger Manager
                      ledgerManager.close();
                      // setting running to false here, so watch thread in bookie server know it only after bookie shut down
                      running = false;
                  }    
              } catch (InterruptedException ie) {
                  LOG.error("Interrupted during shutting down bookie : ", ie); 
              }    
              return exitCode;
          }
      
      1. BK-216.diff
        0.6 kB
        Sijie Guo

        Activity

        Sijie Guo created issue -
        Sijie Guo made changes -
        Field Original Value New Value
        Attachment BK-216.diff [ 12522543 ]
        Sijie Guo made changes -
        Status Open [ 1 ] Patch Available [ 10002 ]
        Ivan Kelly made changes -
        Status Patch Available [ 10002 ] Resolved [ 5 ]
        Resolution Fixed [ 1 ]
        Ivan Kelly made changes -
        Status Resolved [ 5 ] Closed [ 6 ]

          People

          • Assignee:
            Sijie Guo
            Reporter:
            Sijie Guo
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development