You are viewing a plain text version of this content. The canonical link for it is here.

Posted to commits@accumulo.apache.org by me...@apache.org on 2012/09/09 03:13:27 UTC

svn commit: r1382388 - /accumulo/trunk/core/src/main/java/org/apache/accumulo/core/conf/DefaultConfiguration.java

Author: medined
Date: Sun Sep  9 01:13:26 2012
New Revision: 1382388

URL: http://svn.apache.org/viewvc?rev=1382388&view=rev
Log:
ACCUMULO-755: Run findbugs (added synchronized to static method that initializes static variables).

Modified:
    accumulo/trunk/core/src/main/java/org/apache/accumulo/core/conf/DefaultConfiguration.java

Modified: accumulo/trunk/core/src/main/java/org/apache/accumulo/core/conf/DefaultConfiguration.java
URL: http://svn.apache.org/viewvc/accumulo/trunk/core/src/main/java/org/apache/accumulo/core/conf/DefaultConfiguration.java?rev=1382388&r1=1382387&r2=1382388&view=diff
==============================================================================
--- accumulo/trunk/core/src/main/java/org/apache/accumulo/core/conf/DefaultConfiguration.java (original)
+++ accumulo/trunk/core/src/main/java/org/apache/accumulo/core/conf/DefaultConfiguration.java Sun Sep  9 01:13:26 2012
@@ -30,7 +30,7 @@ public class DefaultConfiguration extend
   private static DefaultConfiguration instance = null;
   private static Logger log = Logger.getLogger(DefaultConfiguration.class);
   
-  public static DefaultConfiguration getInstance() {
+  synchronized public static DefaultConfiguration getInstance() {
     if (instance == null) {
       instance = new DefaultConfiguration();
       ConfigSanityCheck.validate(instance);