Skip to content

HTTPS clone URL

Subversion checkout URL

You can clone with HTTPS or Subversion.

Download ZIP

Loading…

Slightly relaxed synchronization #628

Open
wants to merge 1 commit into from

1 participant

@asafdav2

there is no need to hold the lock while querying for the cluster name

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Commits on Jul 21, 2013
  1. @asafdav2
This page is out of date. Refresh to see the latest.
View
2  core/src/main/java/me/prettyprint/hector/api/factory/HFactory.java
@@ -213,8 +213,8 @@ public static void setClusterForTest(Cluster cluster) {
* @param cluster
*/
public static void shutdownCluster(Cluster cluster) {
- synchronized (clusters) {
String clusterName = cluster.getName();
+ synchronized (clusters) {
if (clusters.get(clusterName) != null ) {
cluster.getConnectionManager().shutdown();
clusters.remove(clusterName);
Something went wrong with that request. Please try again.