svn commit: r965151 - in /cassandra/branches/cassandra-0.6: CHANGES.txt src/java/org/apache/cassandra/gms/Gossiper.java

[jbellis]

Author: jbellis
Date: Sun Jul 18 02:05:35 2010
New Revision: 965151

URL: http://svn.apache.org/viewvc?rev=965151&view=rev
Log:
log errors in gossip instead of re-throwing; the latter prevents future gossip 
tasks from running, too.  patch by brandonwilliams and jbellis for 
CASSANDRA-1289

Modified:
    cassandra/branches/cassandra-0.6/CHANGES.txt
    
cassandra/branches/cassandra-0.6/src/java/org/apache/cassandra/gms/Gossiper.java

Modified: cassandra/branches/cassandra-0.6/CHANGES.txt
URL: 
http://svn.apache.org/viewvc/cassandra/branches/cassandra-0.6/CHANGES.txt?rev=965151&r1=965150&r2=965151&view=diff
==============================================================================
--- cassandra/branches/cassandra-0.6/CHANGES.txt (original)
+++ cassandra/branches/cassandra-0.6/CHANGES.txt Sun Jul 18 02:05:35 2010
@@ -8,7 +8,8 @@
  * added describe_partitioner Thrift method (CASSANDRA-1047)
  * Hadoop jobs no longer require the Cassandra storage-conf.xml
    (CASSANDRA-1280, CASSANDRA-1047)
-* log thread pool stats when GC is excessive (CASSANDRA-1275)
+ * log thread pool stats when GC is excessive (CASSANDRA-1275)
+ * log errors in gossip instead of re-throwing (CASSANDRA-1289)
 
 
 0.6.3

Modified: 
cassandra/branches/cassandra-0.6/src/java/org/apache/cassandra/gms/Gossiper.java
URL: 
http://svn.apache.org/viewvc/cassandra/branches/cassandra-0.6/src/java/org/apache/cassandra/gms/Gossiper.java?rev=965151&r1=965150&r2=965151&view=diff
==============================================================================
--- 
cassandra/branches/cassandra-0.6/src/java/org/apache/cassandra/gms/Gossiper.java
 (original)
+++ 
cassandra/branches/cassandra-0.6/src/java/org/apache/cassandra/gms/Gossiper.java
 Sun Jul 18 02:05:35 2010
@@ -96,7 +96,7 @@ public class Gossiper implements IFailur
             }
             catch (Exception e)
             {
-                throw new RuntimeException(e);
+                logger_.error("Gossip error", e);
             }
         }
     }