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

API should sanity check sessionTimeout argument

    Details

    • Type: Improvement
    • Status: Patch Available
    • Priority: Minor
    • Resolution: Unresolved
    • Affects Version/s: 3.2.2, 3.3.0, 3.3.1, 3.4.6, 3.5.0
    • Fix Version/s: 3.6.0, 3.5.5
    • Component/s: c client, java client
    • Labels:
      None
    • Environment:

      OSX 10.6.3, JVM 1.6.0-20

      Description

      passing in a "0" sessionTimeout to ZooKeeper() constructor leads to errors in subsequent operations. It would be ideal to capture this configuration error at the source by throwing something like an IllegalArgument exception when the bogus sessionTimeout is specified, instead of later when it is utilized.

        Attachments

        1. ZOOKEEPER-776.patch
          6 kB
          Raul Gutierrez Segales
        2. zookeeper-776-fix.patch
          2 kB
          Gregory Haskins
        3. zookeeper-776-fix.patch
          3 kB
          Gregory Haskins
        4. zookeeper-776-fix.patch
          1 kB
          Gregory Haskins

          Issue Links

            Activity

              People

              • Assignee:
                rgs Raul Gutierrez Segales
                Reporter:
                ghaskins Gregory Haskins
              • Votes:
                0 Vote for this issue
                Watchers:
                3 Start watching this issue

                Dates

                • Created:
                  Updated: