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

Fix typo

fix typo

CURATOR-547 change JAX-RS reader/writer to reuse Jackson ObjectMapper; also Jackson 2.9.8 -> 2.9.10

Merge pull request #332 from apache/dependabot/maven/jackson-version-2.10.0

Bump jackson-version from 2.9.8 to 2.10.0

Bump jackson-version from 2.9.8 to 2.10.0

Bumps `jackson-version` from 2.9.8 to 2.10.0.

Updates `jackson-core` from 2.9.8 to 2.10.0
- [Release notes](https://github.com/FasterXML/jackson-core/releases)
- [Commits](https://github.com/FasterXML/jackson-core/compare/jackson-core-2.9.8...jackson-core-2.10.0)

Updates `jackson-databind` from 2.9.8 to 2.10.0
- [Release notes](https://github.com/FasterXML/jackson/releases)
- [Commits](https://github.com/FasterXML/jackson/commits)

Updates `jackson-dataformat-yaml` from 2.9.8 to 2.10.0
- [Release notes](https://github.com/FasterXML/jackson-dataformats-text/releases)
- [Commits](https://github.com/FasterXML/jackson-dataformats-text/compare/jackson-dataformats-text-2.9.8...jackson-dataformats-text-2.10.0)

Signed-off-by: dependabot[bot] <support@github.com>

CURATOR-546

Bad copy/paste bug. ModeledCacheImpl.currentData() was removing from entries instead of calling get()

Merge pull request #327 from apache/CURATOR-543-fix-testLockACLs-due-to-ZOOKEEPER-1392

CURATOR-543 - ZOOKEEPER-1392 broke TestLockACLs

CURATOR-543

ZOOKEEPER-1392 broke TestLockACLs. In order to read ACLS you now need READ perm. TestLockACLs.testLockACLs() needs to be updated to reflect this.

Merge branch 'CURATOR-541-x'

temp checking to see if adding synchronized fixes travis

Merge branch 'CURATOR-541'

CURATOR-541

The retry code in BaseClassForTests was hopelessly broken - I don't know for how long. Reworked it so that it does the right thing now (hopefully). Storing the retry count as an attribute wasn't working. The new method stores it as a field in the test class and makes sure that it's always correct. It should only ever be true when actually retrying and, given that TestNG always calls the retry method, it can be reset after a retry fails.

Merge pull request #314 from aseychell/master

CURATOR-519: Update zookeeper to 3.5.5 stable version

Merge pull request #324 from apache/CURATOR-537

[CURATOR-537] make getOurPath() public

Merge pull request #323 from sth/remove-obsolete-link

[CURATOR-539] Remove link to obsolete Stack Overflow tag

CURATOR-537 - make getOurPath() public

Remove link to obsolete Stack Overflow tag

Merge pull request #321 from asdf2014/fix_bytebuffer

CURATOR-523: Fix ByteBuffer's compatibility issues

Merge remote-tracking branch 'upstream/master'

Merge pull request #320 from apache/CURATOR-505-improve-circuit-breaker-to-shared

[CURATOR-533] - improve circuit breaking behavior

CURATOR-533

CURATOR-505 introduced circuit breaking behavior via CircuitBreakingConnectionStateListener and ConnectionStateListenerDecorator. Elastic has been using it to success but reports that the implementation can be improved. The existing implementation uses a new CircuitBreaker for each ConnectionStateListener set in a Curator client. It turns out that this is not ideal. Instead, a shared CircuitBreaker should be used per Curator client.

Unfortunately, the best way to do this is to remove the ConnectionStateListenerDecorator semantics and use a different mechanism. This Issue proposes to do this and remove ConnectionStateListenerDecorator. This is a breaking change but given the short amount of time it's been in Curator it's unlikely that it's been widely adopted.

In this commit, ConnectionStateListenerDecorator is removed in favor of ConnectionStateListenerManagerFactory. ConnectionStateManager uses this factory to create the container to hold registered ConnectionStateListeners. A new CircuitBreakerManager now manages the circuit breaking behavior using a shared CircuitBreaker.

Merge pull request #322 from abcdenis/CURATOR-536

[CURATOR-536] break infinite loop in internalLeave() on timeout

CURATOR-536 break infinite loop in internalLeave() on timeout

subj!

Fix ByteBuffer's compatibility issues

Turn off spammy logs

Switch to OpenJDK 8 and 11

Merge pull request #317 from apache/CURATOR-528-fix-mailing-list-issues

fix mailing list issues

fix mailing-lists.html buttons to have mailto prefix; fix doap.rdf to refer correctly to mailing-lists.html

Merge pull request #315 from emretetik96/CURATOR-530

[CURATOR-530] Documentation on InterProcessSemaphoreMutex is misleading

Add note on InterProcessLock description

Style changes

Modified javadoc for InterProcessLock and subclasses to be less misleading

Update zookeeper to 3.5.5 stable version

Merge pull request #311 from JiriOndrusek/CURATOR-429_osgi-zookeeper-dependency

[CURATOR-429] Make Curator 4.x compatible with Zookeeper 3.4.x in OSGi too

[CURATOR-429] Make Curator 4.x compatible with Zookeeper 3.4.x in OSGi too

added myself in the developers section

fixed some broken left nav links

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

move to gitbox

Merge pull request #305 from BELUGABEHR/CURATOR-511

CURATOR-511: Add toString to ZKPaths PathAndNode

CURATOR-511: Add toString to ZKPaths PathAndNode