CURATOR-630 Upgrade jetty to 9.4.latest (#413)

Set currentStableVersion to 5.2.1

[maven-release-plugin] prepare for next development iteration

[maven-release-plugin] prepare release apache-curator-5.2.1

CURATOR-625 Bump directory maven plugin version to 1.0 (#402)

CURATOR-633: Run TestConnectionStateManager tests in exact groups (#411)

Signed-off-by: tison <wander4096@gmail.com>

CURATOR-624: Fix Github Actions by adding 'distribution' for JDK setup step (#407)

CURATOR-632: Update Maven plugins (#408)

Signed-off-by: Martin Tzvetanov Grigorov <mgrigorov@apache.org>

CURATOR-624 Migrate CI from Travis to GitHub actions (#403)

CURATOR-561 Reset connection after repeat expiry

If there is a problem posting the Expired KeeperState
during a session expiration, then the ZooKeeper event
thread will die without ever posting the Expired event.
This would then cause curator to keep trying to expire
the connection but it does nothing because the connection
is dead and no events will ever be posted.

This can be prevented by forcibly resetting the connection
if it's detected that the previous expiry had no effect

Replace OutstandingOps with JDK bundled Phaser (#365)

CURATOR-606: ModeledFrameworkImpl.update(T model, int version): Use version in all cases. (#393)

CURATOR-607: InterProcessReadWriteLock should expose exposing getLockPath (#394)

Co-authored-by: Nikita Sokolov <faucct@yandex-team.ru>

This commit reverts changes that altered the ZPath parse method based on  incorrect assumptions about what a "resolved" ZPath actually is.

I've added a few notes to the JavaDoc and unit tests to hopefully clear up confusion in this area in the future.

Modified ZPathImpl's "isParameter()" method to properly check for parameters strings (values inside of a pair parameter delimiters).  This corrects issues with checking for path resolution status in certain cases.

Modified ZPathImpl's "resolved()" method to allow for substituted parameters to have leading path separators ("/"). This is useful in the case where a parameter is used for the first node element in the path, eliminating the need for the user to worry about the leading "/".

Added an additional unit test case for the leading "/" changes, and fixed bugs in the basic test cases that were accepting the wrong resolved state of tested ZPaths.

All unit tests for curator pass.

CURATOR-610: Refactor CountCuratorWatcher in TestWatcherIdentity.java …

Fixes [CURATOR-610](https://issues.apache.org/jira/browse/CURATOR-610)

### Description
Refactor test class [CountCuratorWatcher](https://github.com/apache/curator/blob/4a11aaef8b190dc220d35b7a91df294bfa06250e/curator-framework/src/test/java/org/apache/curator/framework/imps/TestWatcherIdentity.java#L42) by using mocking object created by Mockito.

<hr>

##### Key changed/added classes in this PR
- Create mocking object to replace test subclass `CountCuratorWatcher`, decouple test from production code.
- Make test logic more clear by using method stub instead of method overriding.
- Extract AtomicInteger variable out of the test subclass. Use the extracted variable in assertation statement to check `process(WatchedEvent)` method invocation status.

<hr>

Author: wx930910 <wx19930910@gmail.com>

Reviewers: Enrico Olivelli <eolivelli@apache.org>

Closes #397 from wx930910/CURATOR-610

Update currentStableVersion to 5.2.0

[maven-release-plugin] prepare for next development iteration

[maven-release-plugin] prepare release apache-curator-5.2.0

CURATOR-599: Configurable ZookeeperFactory by ZKClientConfig

Option to use ZooKeeper client config.

This seems mandatory for using zookeeper.request.timeout for preventing
the potential race condition of hanging indefinitely, as described at
the ticket.

Author: liran2000 <liran2000@gmail.com>

Reviewers: Enrico Olivelli <eolivelli@apache.org>, Zili Chen, Cameron McKenzie

Closes #391 from liran2000/CURATOR-599

Use Awaitility to instead of Thread sleep method. (#389)

CURATOR-588  Upgrade ZooKeeper to 3.6.3 (#386)

Co-authored-by: Enrico Olivelli <eolivelli@apache.org>

CURATOR-602 fix typo in the variable (#390)

Co-authored-by: Colin Kuo <colin.kuo@ruckuswireless.com>

CURATOR-594: TestingZooKeeperMain isn't setting tickTime, if configured (#383)

CURATOR-591: Update the PersistentNode documentation

As a follow up of https://github.com/apache/curator/pull/380, we should update the `PersistentNode` doc.

Author: Paul Boutes <paul.boutes@gmail.com>

Reviewers: Enrico Olivelli <eolivelli@apache.org>, Zili Chen, Cameron McKenzie

Closes #381 from pboutes/CURATOR-591

CURATOR-526: drop log level for message to debug

ZooKeeper is backwards compatible and accepts older config
string formats, albeit without dynamic configuration
support. Since the older format is acceptable, we should not
log at error when we see the older format and should
instead simply log at debug.

Author: Ammar Khaku <ammar.khaku@gmail.com>

Reviewers: Enrico Olivelli <eolivelli@apache.org>, Russell Bolles

Closes #382 from akhaku/dropLogLevel

CURATOR-590: Add option to disable parent creation for PersistentNode

Adds a `useParentCreation` boolean flag to control the parent creation.

If this flag is set to `false`, the `PersistentNode` won't create the underlying znodes with the `createParentContainersIfNeeded()`, meaning that the parent znodes will have to exist beforehand in order for the `PersistentNode` to succeed its creation.

The `useParentCreation` flag is set to `true` by default.

https://issues.apache.org/jira/browse/CURATOR-590

Author: Paul Boutes <paul.boutes@elastic.co>
Author: Paul Boutes <paul.boutes@gmail.com>

Reviewers: Enrico Olivelli <eolivelli@apache.org>, Cameron McKenzie <mckenzie.cam@gmail.com>, Zili Chen <wander4096@gmail.com>, Jordan Zimmerman <jordan@jordanzimmerman.com>

Closes #380 from pboutes/CURATOR-590

CURATOR-584: Added fault tolerant idempotent Create, SetData and Delete operations

CURATOR-585: fix DiscoveryExample did not check for exceptions

Author: wangjie <wangchenmo1025@gmail.com>

Reviewers: Enrico Olivelli <eolivelli@apache.org>

Closes #375 from iwangjie/CURATOR-585

add workaround

Upgrade to Python 3

CURATOR-576 Import merge script from ZooKeeper project (#369)

Make Enrico Olivelli PMC chair

CURATOR-582: Migrate to jUnit 5.6 (#372)

CURATOR-583: Fix ArrayIndexOutOfBoundsException when passing empty list parameter to reconfigure API

This closes #374 .

CURATOR-581. fix OSGi export pattern

CURATOR-581. fix OSGi export pattern

Add tison as Committer and PMC

Add tison as Committer and PMC

CURATOR-577 - Add a dedicated downloads page to the website
fix downloads page, use Apache Mirros

CURATOR-577 - Add a dedicated downloads page to the website

Add a dedicated downloads page to the website. Note: this page depends
on the new property "currentStableVersion"

[maven-release-plugin] prepare for next development iteration

[maven-release-plugin] prepare release apache.curator-5.1.0

Update README.md

Added Twitter follow badge

CURATOR-575: TestingServer shutdown can cause an NPE

TestingServer shutdown can cause an NPE due to FileTxnSnapLog being closed in a different thread

Author: randgalt <randgalt@apache.org>

Reviewers: Enrico Olivelli <eolivelli@apache.org>, Cameron McKenzie <mckenzie.cam@gmail.com>

Closes #368 from Randgalt/CURATOR-575-fix-testing-server-npe

CURATOR-574 DiscoveryService fatal error on deserializing an empty byte[] as JSON

CURATOR-574 DiscoveryService fatal error on deserializing an empty byte[] as JSON

Merge branch 'master' of https://github.com/apache/curator

Add Enrico Olivelli as Committer and PMC

Merge branch 'master' of https://github.com/apache/curator

Fixed typo

More/better links in the Read Me

Fixed the website badge

[maven-release-plugin] prepare for next development iteration

[maven-release-plugin] prepare release apache-curator-5.0.0

Added some more icons

Added missing headers from CURATOR-569

CURATOR-569

- New methods to handle protected ZNode names

CURATOR-558 was supposed to shade all of Guava - forgot to remove the exclusions for the old guava classes that were still exposed. This was causing the installed JARs to be incorrect (missing the 3 shaded Guava classes) in the client JAR

testDisconnectReconnectEventDoesNotFireValueWatcher() added as part of CURATOR-344 was flakey and would fail on the final assertion of numChangeEvents.get(). I don't believe you can be assured of the exact number so I made it more lenient and hopefully less flakey

make testParentContainerMissing() more resilient by waiting for initial connection

testDeleteChildrenConcurrently() was badly written and error prone. I fixed it so it should run every time now

CURATOR-544 - Fix issue with 'error code' 0 being passed to the KeeperExeception create method. This was breaking a lot of code

JIRA:CURATOR-568

- Javadoc typo

JIRA:CURATOR-568

- Adding ensembleTracker(boolean) and withEnsembleTracker() methods to
CuratorFrameworkFactory.builder() that allows enabling/disabling
ensemble tracking

Document breaking changes

Address comments

CURATOR-544: SessionFailedRetryPolicy

Merge branch 'master' of https://gitbox.apache.org/repos/asf/curator

CURATOR-559 - background thread retries are spoiling the test. Try to work around this

CURATOR-559 - more attempts to keep tests from failing. Make sure count is zeroed after server is stopped.

[CURATOR-464] update classifier and document accordingly

Merge branch 'CURATOR-567-remove-test-clean-state-where-flaky'

CURATOR-567 - At this point, TestCleanState is so flakey we should just turn it off for now. It's not serving much purpose anyway.

Merge branch 'CURATOR-567-remove-test-clean-state-where-flaky'

CURATOR-567 - TestCleanState.closeAndTestClean has meet is desired goal. Start removing it where it causes test flakiness

CURATOR-559 test used InterProcessReadWriteLock - the problem is that the retry can end up in background loop which has its own thread thereby spoiling the test. Instead use a foreground operation for consistent tests

Merge branch 'master' of https://gitbox.apache.org/repos/asf/curator

CURATOR-567 - remove flaky cases of TestCleanState

Merge branch 'master' of github.com:bigmarvin/curator into CURATOR-464

CURATOR-525 - instead of resetting the connection, change the state to RECONNECTED. I'm concerned about LOST/reset loops. This is still a bad hack and needs to be addressed in the future.

CURATOR-525

There is a race whereby the ZooKeeper connection can be healed before Curator is finished processing the new connection state. When this happens
the Curator instance becomes a Zombie stuck in the LOST state. This fix is a "hack". ConnectionStateManager will notice that the connection state is
LOST but that the Curator instance reports that it is connected. When this happens, it is logged and the connection is reset.

For CURATOR-559 make the test a bit more robust. Wait for the connection to go to LOST after server stop

CURATOR-549

Creates a simple bridge that, when using ZK 3.6.0 creates a CuratorCache, and for earlier versions creates a TreeCache. The curator-test-zk35 module ensures that both code paths are tested.

CURATOR-549

The next phase of this issue will implement a bridge cache that bridges TreeCache for pre 3.6 SK and CuratorCache for ZK 3.6+. That bridge will need this TreeCache iterator.

CURATOR-549 (#335)

Adds several recipes that use the new ZOOKEEPER-1416 Persistent Recursive watches from ZooKeeper 3.6.0. PersistentWatcher - A wrapper recipe that keeps a persistent (single or recursive) watch set and active through disconnections, etc. CuratorCache - Completely re-written cache recipe that will replace TreeCache, NodeCache and PathChildrenCache. With the benefit of persistent recursive watchers, the implementation is far simpler, will use significantly less resources and network calls, be easier to support and should be more stable and performant. Wrappers for the older cache's listeners are provided to help with transitions.

Co-authored-by: randgalt <randgalt@apache.org>

[CURATOR-464] attach orignal artifacts with classifier original

CURATOR-564 (#351)

Like was done for TestingServer, catch startup issues for TestingCluster and then re-recreate and re-start the cluster one time. Hopefully this will make the tests more stable.

Co-authored-by: randgalt <randgalt@apache.org>

CURATOR-549 (#334)

Support persistent watchers in ZK 3.6+ while maintaining background compatibility with previous versions of ZK. Adds Curator Framework DSL calls to ZooKeeper's addWatch() method. Subsequent PRs will add recipes. Both the older Framework and the async Framework now have methods to add persistent watchers.

Co-authored-by: randgalt <randgalt@apache.org>

[CURATOR-558] - Updates for ZooKeeper 3.6.0 (#350)

* CURATOR-558

Bring Curator up to ZooKeeper 3.5.6 in preparation for supporting persistent recursive watchers while maintaining background compatability with previous versions of ZK. Added a new module to make sure we maintain compatibility with ZK 3.5.x. ZooKeeper 3.6.0 has some significant changes from previous versions. The reconfig APIs have moved into a new class, ZooKeeperAdmin. This class existed in 3.5.x but wasn't required. Now it is. A bunch of little things changed in the ZK server code which affected Curator's test classes. I moved it all into reflection based calls in Compatibility.java in the test module. We now have modules that test ZK 3.4, 3.5 and 3.6 so we're safe with compatibility. ZooKeeper's MultiTransactionRecord has been removed it seems. That forced CuratorMultiTransactionRecord to be re-written. It's not a public class so hopefully it won't affect anyone.

There is a new module, curator-test-zk35. It forces ZooKeeper 3.5.6 and performs selected tests from the other modules to ensure compatibility. Tests annotated with TestNG groups zk35 and zk35Compatibility are tested. Group zk36 is excluded. Note: these tests will only run from Maven. I don't think IntelliJ/Eclipse support the Maven syntax I used.
Support persistent watchers in ZK 3.6+ while maintaining background compatability with previous versions of ZK. Added a new module to make sure we maintain comaptibility with ZK 3.5.x

* CURATOR-558 - change to version 5.0.0-SNAPSHOT

Co-authored-by: randgalt <randgalt@apache.org>

CURATOR-558 (#344)

Pt1 of change

* Remove the ZK 3.4 compatibility module and code
* Remove the deprecated ListenerContainer that leaks Guava classes into our APIs
* Remove Exhibitor support
* Various minor changes/cleanups

Co-authored-by: randgalt <randgalt@apache.org>

Merge branch 'master' of https://gitbox.apache.org/repos/asf/curator

CURATOR-562 - Remove ConnectionHandlingPolicy - flatten out behavior to match old StandardConnectionHandlingPolicy - closes #348

CURATOR-562 - Remove ConnectionHandlingPolicy - flatten out behavior to match old StandardConnectionHandlingPolicy

CURATOR-559 (#346)

The retry loop mechanism ended up getting nested multiple times causing exponential calls to the retry policy and violating a given policy's limits. Use a thread local to mitigate this so that a retry loop is reused for nested API calls, etc.

CURATOR-560

a) make sure setReuseAddress is set for server when getting a random port
b) always set "tickTime" and "minSessionTimeout" to make our tests run a bit faster
c) fixed incorrect exception message in blockUntilStarted()

[CURATOR-548] Bump zookeeper version to 3.5.7

CURATOR-551 (#345)

Commit 26364c6186fc7c09a9462557b1ca791e9aa70006 (Sat Sep 26 13:13:02 2015) changed HandleHolder.getNewConnectionString() was changed to return the new connection string instead of just a boolean. I believe the value returned should have been ensembleProvider.getConnectionString() not helper.getConnectionString(). TBH I no longer remember the genesis of this change but I can't make the current implementation make sense.

Additionally, a change was made to optionally call zooKeeper.updateServerList(). When this path is taken the handle holder's connection needs to be updated as well or we'll get an infinite loop of changes. The path that runs when ensembleProvider.updateServerListEnabled() is false ends up setting handle holder's connection to ensembleProvider.getConnectionString().

I'm loathe to make such low level changes in code that's existed for a long time. But, my investigation shows that this is how it should be. Hopefully, users can do testing.

Merge pull request #343 from naude-r/servicecache_thread_leak

[CURATOR-557] ServiceCacheImpl does not close ExecutorService

- ServiceCacheImpl should allow the self-created executor service to be closed
- Allow an optional executor service for ServiceProvider