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 02:49:39 UTC
svn commit: r1382386 -
/accumulo/trunk/core/src/main/java/org/apache/accumulo/core/conf/SiteConfiguration.java
Author: medined
Date: Sun Sep 9 00:49:39 2012
New Revision: 1382386
URL: http://svn.apache.org/viewvc?rev=1382386&view=rev
Log:
ACCUMULO-755: Run findbugs (added synchronized to static methods that initialize static variables).
Modified:
accumulo/trunk/core/src/main/java/org/apache/accumulo/core/conf/SiteConfiguration.java
Modified: accumulo/trunk/core/src/main/java/org/apache/accumulo/core/conf/SiteConfiguration.java
URL: http://svn.apache.org/viewvc/accumulo/trunk/core/src/main/java/org/apache/accumulo/core/conf/SiteConfiguration.java?rev=1382386&r1=1382385&r2=1382386&view=diff
==============================================================================
--- accumulo/trunk/core/src/main/java/org/apache/accumulo/core/conf/SiteConfiguration.java (original)
+++ accumulo/trunk/core/src/main/java/org/apache/accumulo/core/conf/SiteConfiguration.java Sun Sep 9 00:49:39 2012
@@ -35,7 +35,7 @@ public class SiteConfiguration extends A
SiteConfiguration.parent = parent;
}
- public static SiteConfiguration getInstance(AccumuloConfiguration parent) {
+ synchronized public static SiteConfiguration getInstance(AccumuloConfiguration parent) {
if (instance == null) {
instance = new SiteConfiguration(parent);
ConfigSanityCheck.validate(instance);
@@ -43,7 +43,7 @@ public class SiteConfiguration extends A
return instance;
}
- private static Configuration getXmlConfig() {
+ synchronized private static Configuration getXmlConfig() {
String configFile = System.getProperty("org.apache.accumulo.config.file", "accumulo-site.xml");
if (xmlConfig == null) {
xmlConfig = new Configuration(false);