svn commit: r692335 - in /hadoop/core/trunk: CHANGES.txt src/core/org/apache/hadoop/net/NetworkTopology.java

[omalley]

Author: omalley
Date: Thu Sep  4 21:52:16 2008
New Revision: 692335

URL: http://svn.apache.org/viewvc?rev=692335&view=rev
Log:
HADOOP-3943. Remove unnecessary synchronization in 
NetworkTopology.pseudoSortByDistance. (hairong via omalley)

Modified:
    hadoop/core/trunk/CHANGES.txt
    hadoop/core/trunk/src/core/org/apache/hadoop/net/NetworkTopology.java

Modified: hadoop/core/trunk/CHANGES.txt
URL: 
http://svn.apache.org/viewvc/hadoop/core/trunk/CHANGES.txt?rev=692335&r1=692334&r2=692335&view=diff
==============================================================================
--- hadoop/core/trunk/CHANGES.txt (original)
+++ hadoop/core/trunk/CHANGES.txt Thu Sep  4 21:52:16 2008
@@ -264,6 +264,9 @@
     HADOOP-3937. Limit the job name in the job history filename to 50 
     characters. (Matei Zaharia via omalley)
 
+    HADOOP-3943. Remove unnecessary synchronization in 
+    NetworkTopology.pseudoSortByDistance. (hairong via omalley)
+
   OPTIMIZATIONS
 
     HADOOP-3556. Removed lock contention in MD5Hash by changing the 

Modified: hadoop/core/trunk/src/core/org/apache/hadoop/net/NetworkTopology.java
URL: 
http://svn.apache.org/viewvc/hadoop/core/trunk/src/core/org/apache/hadoop/net/NetworkTopology.java?rev=692335&r1=692334&r2=692335&view=diff
==============================================================================
--- hadoop/core/trunk/src/core/org/apache/hadoop/net/NetworkTopology.java 
(original)
+++ hadoop/core/trunk/src/core/org/apache/hadoop/net/NetworkTopology.java Thu 
Sep  4 21:52:16 2008
@@ -616,7 +616,7 @@
    * location at postion 0.
    * It leaves the rest nodes untouched.
    */
-  public synchronized void pseudoSortByDistance( Node reader, Node[] nodes ) {
+  public void pseudoSortByDistance( Node reader, Node[] nodes ) {
     int tempIndex = 0;
     if (reader != null ) {
       int localRackNode = -1;