See <https://builds.apache.org/job/Curator-3.0/org.apache.curator$curator-client/10/changes>
Changes: [randgalt] Continued work on new LOST behavior. Added some tests. To get correct behavior it's necessary to not retry connection failures. Retrying connection failures was never a good idea and here's a good opportunity to fix it as this requires client action to enable [randgalt] minor reformat [randgalt] wip [randgalt] Fixed testRetry() for new LOST behavior [randgalt] removed some test code [randgalt] major refactoring. Abstracting old/new behavior into a pluggable ConnectionHandlingPolicy. Also, IMPORTANT, made the new behavior the default. This needs to be discussed but it's a major improvement and we should default to it. [randgalt] When the connection timeout elapses and there is more than one server in the connection string, reset the connection and try again [randgalt] More refinement of classic/new connection handling. Reworked how the retry policy is invoked for each. New behavior is now confirmed to be: wait for connection timeout only once. Some tests will need work due to this [randgalt] more work on repairing tests for new connection handling [randgalt] doc [randgalt] The true session timeout is a negotiated value between client and server. The new session timeout handling should use the negotiated value if available [randgalt] Use the negotiated session timeout in checkTimeouts() ------------------------------------------ [INFO] [INFO] ------------------------------------------------------------------------ [INFO] Building Curator Client 3.0.0-SNAPSHOT [INFO] ------------------------------------------------------------------------ [INFO] [INFO] --- maven-clean-plugin:2.5:clean (default-clean) @ curator-client --- [INFO] [INFO] --- maven-remote-resources-plugin:1.5:process (default) @ curator-client --- [INFO] [INFO] --- maven-resources-plugin:2.6:resources (default-resources) @ curator-client --- [INFO] Using 'UTF-8' encoding to copy filtered resources. [INFO] Copying 2 resources to META-INF [INFO] Copying 3 resources [INFO] [INFO] --- maven-compiler-plugin:3.1:compile (default-compile) @ curator-client --- [INFO] Changes detected - recompiling the module! [INFO] Compiling 40 source files to <https://builds.apache.org/job/Curator-3.0/org.apache.curator$curator-client/ws/target/classes> [INFO] [INFO] >>> clirr-maven-plugin:2.6.1:check (default) > compile @ curator-client >>> [INFO] [INFO] --- maven-remote-resources-plugin:1.5:process (default) @ curator-client --- [INFO] [INFO] --- maven-resources-plugin:2.6:resources (default-resources) @ curator-client --- [INFO] Using 'UTF-8' encoding to copy filtered resources. [INFO] Copying 2 resources to META-INF [INFO] Copying 3 resources [INFO] Copying 3 resources [INFO] [INFO] --- maven-compiler-plugin:3.1:compile (default-compile) @ curator-client --- [INFO] Nothing to compile - all classes are up to date [WARNING] Failed to getClass for org.codehaus.mojo.clirr.ClirrCheckMojo [CHECKSTYLE] No report found for mojo check[INFO] [INFO] <<< clirr-maven-plugin:2.6.1:check (default) < compile @ curator-client <<< [INFO] [INFO] --- clirr-maven-plugin:2.6.1:check (default) @ curator-client --- [INFO] artifact org.apache.curator:curator-client: checking for updates from Nexus [INFO] artifact org.apache.curator:curator-client: checking for updates from central [INFO] Comparing to version: 2.8.0 [INFO] Succeeded with 0 errors; 0 warnings; and 0 other changes. [CHECKSTYLE] No report found for mojo check[INFO] [INFO] --- maven-bundle-plugin:2.3.7:manifest (bundle-manifest) @ curator-client --- [INFO] [INFO] --- maven-resources-plugin:2.6:testResources (default-testResources) @ curator-client --- [INFO] Using 'UTF-8' encoding to copy filtered resources. [INFO] Copying 1 resource [INFO] Copying 3 resources [INFO] [INFO] --- maven-compiler-plugin:3.1:testCompile (default-testCompile) @ curator-client --- [INFO] Changes detected - recompiling the module! [INFO] Compiling 8 source files to <https://builds.apache.org/job/Curator-3.0/org.apache.curator$curator-client/ws/target/test-classes> [WARNING] <https://builds.apache.org/job/Curator-3.0/org.apache.curator$curator-client/ws/src/test/java/org/apache/curator/TestEnsurePath.java>: <https://builds.apache.org/job/Curator-3.0/org.apache.curator$curator-client/ws/src/test/java/org/apache/curator/TestEnsurePath.java> uses or overrides a deprecated API. [WARNING] <https://builds.apache.org/job/Curator-3.0/org.apache.curator$curator-client/ws/src/test/java/org/apache/curator/TestEnsurePath.java>: Recompile with -Xlint:deprecation for details. [INFO] [INFO] --- maven-surefire-plugin:2.17:test (default-test) @ curator-client --- [INFO] Surefire report directory: <https://builds.apache.org/job/Curator-3.0/org.apache.curator$curator-client/ws/target/surefire-reports> ------------------------------------------------------- T E S T S ------------------------------------------------------- ------------------------------------------------------- T E S T S ------------------------------------------------------- Running org.apache.curator.TestSessionFailRetryLoop Tests run: 8, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 21.533 sec - in org.apache.curator.TestSessionFailRetryLoop Running org.apache.curator.ensemble.exhibitor.TestExhibitorEnsembleProvider Tests run: 6, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 10.683 sec - in org.apache.curator.ensemble.exhibitor.TestExhibitorEnsembleProvider Running org.apache.curator.utils.TestZKPaths Tests run: 2, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 0.415 sec - in org.apache.curator.utils.TestZKPaths Running org.apache.curator.utils.TestCloseableScheduledExecutorService Tests run: 2, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 2.449 sec - in org.apache.curator.utils.TestCloseableScheduledExecutorService Running org.apache.curator.utils.TestCloseableExecutorService Tests run: 5, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 0.487 sec - in org.apache.curator.utils.TestCloseableExecutorService Running org.apache.curator.TestRetryLoop Tests run: 8, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 27.934 sec - in org.apache.curator.TestRetryLoop Running org.apache.curator.TestEnsurePath Tests run: 2, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 0.657 sec - in org.apache.curator.TestEnsurePath Results : Tests run: 33, Failures: 0, Errors: 0, Skipped: 0 [JENKINS] Recording test results log4j:WARN No appenders could be found for logger (org.apache.commons.beanutils.converters.BooleanConverter). log4j:WARN Please initialize the log4j system properly. [INFO] [INFO] --- maven-bundle-plugin:2.3.7:bundle (default-bundle) @ curator-client --- [INFO] [INFO] --- maven-site-plugin:3.4:attach-descriptor (attach-descriptor) @ curator-client --- [INFO] [INFO] --- maven-license-plugin:1.9.0:check (license) @ curator-client --- [INFO] Checking licenses... [INFO] Missing header in: <https://builds.apache.org/job/Curator-3.0/org.apache.curator$curator-client/ws/src/main/java/org/apache/curator/connection/ClassicConnectionHandlingPolicy.java> [INFO] Missing header in: <https://builds.apache.org/job/Curator-3.0/org.apache.curator$curator-client/ws/src/main/java/org/apache/curator/connection/ConnectionHandlingPolicy.java> [INFO] Missing header in: <https://builds.apache.org/job/Curator-3.0/org.apache.curator$curator-client/ws/src/main/java/org/apache/curator/connection/StandardConnectionHandlingPolicy.java> [CHECKSTYLE] No report found for mojo check
