Uploaded image for project: 'ZooKeeper'
  1. ZooKeeper
  2. ZOOKEEPER-575

remove System.exit calls to make the server more container friendly

    XMLWordPrintableJSON

Details

    • Improvement
    • Status: Open
    • Major
    • Resolution: Unresolved
    • 3.4.0
    • None
    • server
    • None

    Description

      There are a handful of places left in the code that still use System.exit, we should remove these to make the server
      more container friendly.

      There are some legitimate places for the exits - in *Main.java for example should be fine - these are the command
      line main routines. Containers should be embedding code that runs just below this layer (or we should refactor
      so that it would).

      The tricky bit is ensuring the server shuts down in case of an unrecoverable error occurring, afaik these are the
      locations where we still have sys exit calls.

      Attachments

        1. ZOOKEEPER-575.patch
          32 kB
          Andrew Finnell
        2. ZOOKEEPER-575-2.patch
          2 kB
          Andrew Finnell
        3. ZOOKEEPER-575-3.patch
          32 kB
          Andrew Finnell
        4. ZOOKEEPER-575_4.patch
          24 kB
          Mahadev Konar

        Issue Links

          is depended upon by

          Task - A task that needs to be done. ZOOKEEPER-1072 Support for embedded ZooKeeper

          • Major - Major loss of function.
          • Open

          Activity

            People

              andrew.finnell Andrew Finnell
              phunt Patrick D. Hunt
              Votes:
              2 Vote for this issue
              Watchers:
              6 Start watching this issue

              Dates

                Created:
                Updated: