You are viewing a plain text version of this content. The canonical link for it is here.
Posted to common-commits@hadoop.apache.org by cd...@apache.org on 2009/11/21 05:22:15 UTC
svn commit: r882832 - in /hadoop/common/branches/branch-0.21: CHANGES.txt
src/java/org/apache/hadoop/conf/Configuration.java
Author: cdouglas
Date: Sat Nov 21 04:22:15 2009
New Revision: 882832
URL: http://svn.apache.org/viewvc?rev=882832&view=rev
Log:
HADOOP-6269. Fix threading issue with defaultResource in Configuration.
Contributed by Sreekanth Ramakrishnan
Modified:
hadoop/common/branches/branch-0.21/CHANGES.txt
hadoop/common/branches/branch-0.21/src/java/org/apache/hadoop/conf/Configuration.java
Modified: hadoop/common/branches/branch-0.21/CHANGES.txt
URL: http://svn.apache.org/viewvc/hadoop/common/branches/branch-0.21/CHANGES.txt?rev=882832&r1=882831&r2=882832&view=diff
==============================================================================
--- hadoop/common/branches/branch-0.21/CHANGES.txt (original)
+++ hadoop/common/branches/branch-0.21/CHANGES.txt Sat Nov 21 04:22:15 2009
@@ -1134,6 +1134,9 @@
count at the start of each block in Hadoop archives. (Ben Slusky, Tom
White, and Mahadev Konar via cdouglas)
+ HADOOP-6269. Fix threading issue with defaultResource in Configuration.
+ (Sreekanth Ramakrishnan via cdouglas)
+
Release 0.20.1 - 2009-09-01
INCOMPATIBLE CHANGES
Modified: hadoop/common/branches/branch-0.21/src/java/org/apache/hadoop/conf/Configuration.java
URL: http://svn.apache.org/viewvc/hadoop/common/branches/branch-0.21/src/java/org/apache/hadoop/conf/Configuration.java?rev=882832&r1=882831&r2=882832&view=diff
==============================================================================
--- hadoop/common/branches/branch-0.21/src/java/org/apache/hadoop/conf/Configuration.java (original)
+++ hadoop/common/branches/branch-0.21/src/java/org/apache/hadoop/conf/Configuration.java Sat Nov 21 04:22:15 2009
@@ -44,6 +44,7 @@
import java.util.Set;
import java.util.StringTokenizer;
import java.util.WeakHashMap;
+import java.util.concurrent.CopyOnWriteArrayList;
import java.util.regex.Matcher;
import java.util.regex.Pattern;
@@ -168,8 +169,8 @@
* List of default Resources. Resources are loaded in the order of the list
* entries
*/
- private static final ArrayList<String> defaultResources =
- new ArrayList<String>();
+ private static final CopyOnWriteArrayList<String> defaultResources =
+ new CopyOnWriteArrayList<String>();
private static final Map<ClassLoader, Map<String, Class<?>>>
CACHE_CLASSES = new WeakHashMap<ClassLoader, Map<String, Class<?>>>();
@@ -1693,7 +1694,7 @@
return sb.toString();
}
- private void toString(ArrayList resources, StringBuffer sb) {
+ private void toString(List resources, StringBuffer sb) {
ListIterator i = resources.listIterator();
while (i.hasNext()) {
if (i.nextIndex() != 0) {