Skip to content

HTTPS clone URL

Subversion checkout URL

You can clone with
or
.
Download ZIP

Loading…

Change CassandraDaemon to decrement latch after finished with Cassandra s #120

Merged
merged 1 commit into from

2 participants

@cnauroth

Hi tjake,

I'm Chris Nauroth. I'm a software engineer at Disney working on a project that integrates with Solandra. I think you've spoken with Arun Jacob. I'm a colleague of his. We're really excited about Solandra, and it's been working great for us so far in a test environment.

I'd like to submit a few patches for your consideration. This is the first one. CassandraUtils.startupServer starts a CassandraDaemon, and then waits on a CountDownLatch for notification that Cassandra initialization has completed. I have some automated integration tests that were sporadically returning from CassandraUtils.startupServer before Cassandra startup had really finished. This patch fixes the issue by reordering some code in CassandraDaemon.java so that it decrements the latch after Cassandra completes initialization (not before).

Thanks,
--Chris

@tjake tjake merged commit 3e70885 into tjake:solandra
@tjake
Owner

Thanks Chris!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Commits on Aug 17, 2011
  1. @cnauroth
This page is out of date. Refresh to see the latest.
Showing with 1 addition and 4 deletions.
  1. +1 −4 src/lucandra/CassandraDaemon.java
View
5 src/lucandra/CassandraDaemon.java
@@ -28,11 +28,8 @@
public void start()
{
-
- startedLatch.countDown();
-
- // TODO Auto-generated method stub
super.start();
+ startedLatch.countDown();
}
public CountDownLatch getStartedLatch()
Something went wrong with that request. Please try again.