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

handle errors when parsing config file, throw illegalargumentexception rather than exit()

    XMLWordPrintableJSON

Details

    • Improvement
    • Status: Closed
    • Minor
    • Resolution: Fixed
    • 3.0.0
    • 3.0.1, 3.1.0
    • server
    • None
    • Reviewed

    Description

      Discussing 209 with Ben today, we thought that it would be better to have the parse method of QuorumPeerConfig returning a boolean that indicates whether the configuration is good or not, and let the caller decide whether to exit or not. Currently we execute a System.exit() on QuorumPeerConfig.parse when we have a critical configuration error.

      Attachments

        1. ZOOKEEPER-217.patch
          17 kB
          Mahadev Konar
        2. ZOOKEEPER-217.patch
          16 kB
          Mahadev Konar

        Issue Links

          Activity

            People

              mahadev Mahadev Konar
              fpj Flavio Paiva Junqueira
              Votes:
              0 Vote for this issue
              Watchers:
              0 Start watching this issue

              Dates

                Created:
                Updated:
                Resolved: