Author: jbellis
Date: Mon Oct 12 19:05:40 2009
New Revision: 824458
URL: http://svn.apache.org/viewvc?rev=824458&view=rev
Log:
fix race condition in ResponseVerbHandler. patch by jbellis; reviewed by
eevans for CASSANDRA-478
Modified:
incubator/cassandra/branches/cassandra-0.4/CHANGES.txt
incubator/cassandra/branches/cassandra-0.4/src/java/org/apache/cassandra/service/ConsistencyManager.java
Modified: incubator/cassandra/branches/cassandra-0.4/CHANGES.txt
URL:
http://svn.apache.org/viewvc/incubator/cassandra/branches/cassandra-0.4/CHANGES.txt?rev=824458&r1=824457&r2=824458&view=diff
==============================================================================
--- incubator/cassandra/branches/cassandra-0.4/CHANGES.txt (original)
+++ incubator/cassandra/branches/cassandra-0.4/CHANGES.txt Mon Oct 12 19:05:40
2009
@@ -8,6 +8,7 @@
* Add SnapshotBeforeCompaction configuration option (CASSANDRA-426)
* Fix compaction abort under insufficient disk space (CASSANDRA-473)
* Fix reading subcolumn slice from tombstoned CF (CASSANDRA-484)
+ * Fix race condition in RVH causing occasional NPE (CASSANDRA-478)
0.4.0
Modified:
incubator/cassandra/branches/cassandra-0.4/src/java/org/apache/cassandra/service/ConsistencyManager.java
URL:
http://svn.apache.org/viewvc/incubator/cassandra/branches/cassandra-0.4/src/java/org/apache/cassandra/service/ConsistencyManager.java?rev=824458&r1=824457&r2=824458&view=diff
==============================================================================
---
incubator/cassandra/branches/cassandra-0.4/src/java/org/apache/cassandra/service/ConsistencyManager.java
(original)
+++
incubator/cassandra/branches/cassandra-0.4/src/java/org/apache/cassandra/service/ConsistencyManager.java
Mon Oct 12 19:05:40 2009
@@ -48,7 +48,7 @@
{
List<Message> responses_ = new ArrayList<Message>();
- public void response(Message msg)
+ public synchronized void response(Message msg)
{
responses_.add(msg);
if ( responses_.size() ==
ConsistencyManager.this.replicas_.size() )