Author: jbellis
Date: Fri Aug  7 17:03:32 2009
New Revision: 802092

URL: http://svn.apache.org/viewvc?rev=802092&view=rev
Log:
human-readable error for bad CommitLogSyncDelay.
patch by jbellis; reviewed by Eric Evans for CASSANDRA-349

Modified:
    
incubator/cassandra/trunk/src/java/org/apache/cassandra/config/DatabaseDescriptor.java

Modified: 
incubator/cassandra/trunk/src/java/org/apache/cassandra/config/DatabaseDescriptor.java
URL: 
http://svn.apache.org/viewvc/incubator/cassandra/trunk/src/java/org/apache/cassandra/config/DatabaseDescriptor.java?rev=802092&r1=802091&r2=802092&view=diff
==============================================================================
--- 
incubator/cassandra/trunk/src/java/org/apache/cassandra/config/DatabaseDescriptor.java
 (original)
+++ 
incubator/cassandra/trunk/src/java/org/apache/cassandra/config/DatabaseDescriptor.java
 Fri Aug  7 17:03:32 2009
@@ -144,9 +144,16 @@
                 // Bool.valueOf will silently assume false for values it 
doesn't recognize
                 throw new ConfigurationException("Unrecognized value for 
CommitLogSync.  Use 'true' or 'false'.");
             }
-            commitLogSync_ = 
Boolean.valueOf(xmlUtils.getNodeValue("/Storage/CommitLogSync"));
+            commitLogSync_ = Boolean.valueOf(syncRaw);
 
-            commitLogSyncDelay_ = 
Integer.valueOf(xmlUtils.getNodeValue("/Storage/CommitLogSyncDelay"));
+            try
+            {
+                commitLogSyncDelay_ = 
Integer.valueOf(xmlUtils.getNodeValue("/Storage/CommitLogSyncDelay"));
+            }
+            catch (Exception e)
+            {
+                throw new ConfigurationException("Unrecognized value for 
CommitLogSyncDelay.  Integer expected.");
+            }
 
             /* Hashing strategy */
             String partitionerClassName = 
xmlUtils.getNodeValue("/Storage/Partitioner");


Reply via email to