[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

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

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

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

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

Merge pull request #259 from krichter722/license-plugin-phase

changed phase of maven-license-plugin to validate

Merge branch 'CURATOR-481' of github.com:Max-Pudov/curator into CURATOR-481

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

CURATOR-503 guava has been broken into multiple JARs it seems. Our shading needs to reflect that

Merge branch 'update-deps-2018' of github.com:metamx/curator into CURATOR-503

Update README.md

Update README.md

Closes CURATOR-413 - Added travis config file

Merge branch 'CURATOR-505'

CURATOR-505 - fix elapsed calculation in the circuit breaker

CURATOR-505 - didn't mean to check this in

Merge branch 'CURATOR-498'

CURATOR-505 - Some refactoring and more doc

CURATOR-505 ctor should be private

CURATOR-505 - refactoring/refining a new listener container that doesn't rely on Guava and supports mapping. We need for this PR anyway.

CURATOR-505 - decoration of ConnectionStateListeners is now automatic (a backdoor is provided)

CURATOR-505 - interim checking - refactoring, simplifications, more testing, and documentation

CURATOR-505

A decorator/proxy for connection state listeners that adds circuit breaking behavior. During network outages ZooKeeper can become very noisy sending connection/disconnection events in rapid succession. Curator recipes respond to these messages by resetting state, etc. E.g. LeaderLatch must delete its lock node and try to recreated it in order to try to re-obtain leadership, etc.

This noisy herding can be avoided by using the circuit breaking listener decorator. When it receives ConnectionState.SUSPENDED, the circuit becomes "open" (based on the provided RetryPolicy) and will ignore future connection state changes until RetryPolicy timeout has elapsed. Note: however, if the connection goes from ConnectionState.SUSPENDED to ConnectionState.LOST the first LOST state is sent.

Improve comment

CURATOR-498 - minor tuning to testSessionSurvives() test

CURATOR-498

change to how session expiration is injected introduced flakiness into testSessionConnectionStateErrorPolicyWithExpirationPercent30. I've tightened up the test.

CURATOR-498

Tightened LeaderLatch's timed await method so it's more consistent. The two calls to hasLeadership() were affecting the testWatchedNodeDeletedOnReconnect() test. In practice this is likely not an issue, but it's more consistent now. Also, reworked the testWatchedNodeDeletedOnReconnect() a bit.

CURATOR-498

Removed no-longer-necessary KillSession2. Also, now always use the reflection based code to insert the session end event given what we've found about the previous version inside of ZooKeeper. I also opened a PR in ZooKeeper (see ZOOKEEPER-3269) to add a supported method to do this for the future.

CURATOR-500 - move to Gitbox

CURATOR-498

Missed a check on doProtected in checkSetSessionId()

CURATOR-498

1. Abstracted protected mode fields into a new class for clarity. 2. CURATOR-498's fix is making TestTreeCache.testKilledSession fail for zk 3.4 emulation mode. It's strange, but simple to work around. Should likely be revisited in the future.

CURATOR-498

Kudos to user Shay Shimony for his tireless and excellent work tracking this down. There are two problems addressed here: 1) Protected create mode can potentially find a ZNode that is about to be deleted due to an expired session. CreateBuilderImpl now keeps track of the session ID when the create is initiated. If after a connection loss the session ID has changed, any found protected node is ignored as it will soon be deleted. 2) For ZooKeeper 3.4.x the simulated (via reflection) InjectSessionExpiration was incorrectly setting the connection state to closed which caused the session expiration to be ignored.

Update deps

CURATOR-501 remove ZK 3.5 only class from expectedExceptions in test annotation

CURATOR-497 - include a link to the Download wiki page and update doxia revs

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

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

CURATOR-495 runSafeService should return this

CURATOR-495 - have methods return a CompletableFuture so that callers can check completion, etc.

CURATOR-495

Fixes race in many Curator recipes whereby a pattern was used that called "notifyAll" in a synchronized block in response to a ZooKeeper watcher callback. This created a race and possible deadlock if the recipe instance was already in a synchronized block. This would result in the ZK event thread getting blocked which would prevent ZK connections from getting repaired. This change adds a new executor (available from CuratorFramework) that can be used to do the sync/notify so that ZK's event thread is not blocked.

Merge branch 'CURATOR-479-2'

Merge branch 'CURATOR-479-2' of https://git-wip-us.apache.org/repos/asf/curator into CURATOR-479-2

CURATOR-479

CachedModeledFrameworkImpl.children() and CachedModeledFrameworkImpl.childrenAsZNodes() were not implemented correctly but neither was the previous commit for this. This is the correct fix. Previous commit was returning full tree, should be 1 level only

CURATOR-479

CachedModeledFrameworkImpl.children() and CachedModeledFrameworkImpl.childrenAsZNodes() were not implemented correctly but neither was the previous commit for this. This is the correct fix. Previous commit was returning full tree, should be 1 level only

Make ZKPaths.makePath() methods to allocate less garbage

Merge branch 'master' of https://git-wip-us.apache.org/repos/asf/curator

Merge branch 'CURATOR-479'