ZooKeeper
  1. ZooKeeper
  2. ZOOKEEPER-490

the java docs for session creation are misleading/incomplete

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: 3.1.1, 3.2.0
    • Fix Version/s: 3.2.1, 3.3.0
    • Component/s: None
    • Labels:
      None
    • Hadoop Flags:
      Reviewed

      Description

      the javadoc for ZooKeeper constructor says:

      • The client object will pick an arbitrary server and try to connect to it.
      • If failed, it will try the next one in the list, until a connection is
      • established, or all the servers have been tried.

      the "or all server tried" phrase is misleading, it should indicate that we retry until success, con closed, or session expired.

      we also need ot mention that connection is async, that constructor returns immed and you need to look for connection event in watcher

        Activity

        Transition Time In Source Status Execution Times Last Executer Last Execution Date
        Open Open Patch Available Patch Available
        7d 4h 38m 1 Patrick Hunt 05/Aug/09 23:06
        Patch Available Patch Available Resolved Resolved
        1d 19h 11m 1 Patrick Hunt 07/Aug/09 18:17
        Resolved Resolved Closed Closed
        29d 4h 18m 1 Patrick Hunt 05/Sep/09 22:36
        Patrick Hunt made changes -
        Status Resolved [ 5 ] Closed [ 6 ]
        Hide
        Hudson added a comment -

        Integrated in ZooKeeper-trunk #409 (See http://hudson.zones.apache.org/hudson/job/ZooKeeper-trunk/409/)
        . the java docs for session creation are misleading/incomplete

        Show
        Hudson added a comment - Integrated in ZooKeeper-trunk #409 (See http://hudson.zones.apache.org/hudson/job/ZooKeeper-trunk/409/ ) . the java docs for session creation are misleading/incomplete
        Patrick Hunt made changes -
        Status Patch Available [ 10002 ] Resolved [ 5 ]
        Resolution Fixed [ 1 ]
        Hide
        Patrick Hunt added a comment -

        I've committed based on ben's +1.

        Show
        Patrick Hunt added a comment - I've committed based on ben's +1.
        Hide
        Hadoop QA added a comment -

        -1 overall. Here are the results of testing the latest attachment
        http://issues.apache.org/jira/secure/attachment/12415675/ZOOKEEPER-490.patch
        against trunk revision 801839.

        +1 @author. The patch does not contain any @author tags.

        -1 tests included. The patch doesn't appear to include any new or modified tests.
        Please justify why no tests are needed for this patch.

        +1 javadoc. The javadoc tool did not generate any warning messages.

        +1 javac. The applied patch does not increase the total number of javac compiler warnings.

        +1 findbugs. The patch does not introduce any new Findbugs warnings.

        +1 release audit. The applied patch does not increase the total number of release audit warnings.

        +1 core tests. The patch passed core unit tests.

        +1 contrib tests. The patch passed contrib unit tests.

        Test results: http://hudson.zones.apache.org/hudson/job/Zookeeper-Patch-vesta.apache.org/174/testReport/
        Findbugs warnings: http://hudson.zones.apache.org/hudson/job/Zookeeper-Patch-vesta.apache.org/174/artifact/trunk/build/test/findbugs/newPatchFindbugsWarnings.html
        Console output: http://hudson.zones.apache.org/hudson/job/Zookeeper-Patch-vesta.apache.org/174/console

        This message is automatically generated.

        Show
        Hadoop QA added a comment - -1 overall. Here are the results of testing the latest attachment http://issues.apache.org/jira/secure/attachment/12415675/ZOOKEEPER-490.patch against trunk revision 801839. +1 @author. The patch does not contain any @author tags. -1 tests included. The patch doesn't appear to include any new or modified tests. Please justify why no tests are needed for this patch. +1 javadoc. The javadoc tool did not generate any warning messages. +1 javac. The applied patch does not increase the total number of javac compiler warnings. +1 findbugs. The patch does not introduce any new Findbugs warnings. +1 release audit. The applied patch does not increase the total number of release audit warnings. +1 core tests. The patch passed core unit tests. +1 contrib tests. The patch passed contrib unit tests. Test results: http://hudson.zones.apache.org/hudson/job/Zookeeper-Patch-vesta.apache.org/174/testReport/ Findbugs warnings: http://hudson.zones.apache.org/hudson/job/Zookeeper-Patch-vesta.apache.org/174/artifact/trunk/build/test/findbugs/newPatchFindbugsWarnings.html Console output: http://hudson.zones.apache.org/hudson/job/Zookeeper-Patch-vesta.apache.org/174/console This message is automatically generated.
        Benjamin Reed made changes -
        Hadoop Flags [Reviewed]
        Hide
        Benjamin Reed added a comment -

        +1 looks good pat

        Show
        Benjamin Reed added a comment - +1 looks good pat
        Patrick Hunt made changes -
        Status Open [ 1 ] Patch Available [ 10002 ]
        Patrick Hunt made changes -
        Attachment ZOOKEEPER-490.patch [ 12415675 ]
        Hide
        Patrick Hunt added a comment -

        this patch updates the javadoc for zk construction
        talks about async nature
        talks about thread safety

        Show
        Patrick Hunt added a comment - this patch updates the javadoc for zk construction talks about async nature talks about thread safety
        Patrick Hunt made changes -
        Field Original Value New Value
        Assignee Patrick Hunt [ phunt ]
        Patrick Hunt created issue -

          People

          • Assignee:
            Patrick Hunt
            Reporter:
            Patrick Hunt
          • Votes:
            0 Vote for this issue
            Watchers:
            1 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development