db-derby-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From kahat...@apache.org
Subject svn commit: r1342566 - /db/derby/code/trunk/java/engine/org/apache/derby/impl/store/access/RAMTransaction.java
Date Fri, 25 May 2012 10:44:39 GMT
Author: kahatlen
Date: Fri May 25 10:44:38 2012
New Revision: 1342566

URL: http://svn.apache.org/viewvc?rev=1342566&view=rev
Log:
DERBY-3371: Remove unnecessary remove() before put()
in RAMTransaction.addColumnToConglomerate()

Map.put() replaces the old mapping for the key, if there is one, so
there is no need to remove it first.

Modified:
    db/derby/code/trunk/java/engine/org/apache/derby/impl/store/access/RAMTransaction.java

Modified: db/derby/code/trunk/java/engine/org/apache/derby/impl/store/access/RAMTransaction.java
URL: http://svn.apache.org/viewvc/db/derby/code/trunk/java/engine/org/apache/derby/impl/store/access/RAMTransaction.java?rev=1342566&r1=1342565&r2=1342566&view=diff
==============================================================================
--- db/derby/code/trunk/java/engine/org/apache/derby/impl/store/access/RAMTransaction.java
(original)
+++ db/derby/code/trunk/java/engine/org/apache/derby/impl/store/access/RAMTransaction.java
Fri May 25 10:44:38 2012
@@ -621,9 +621,6 @@ public class RAMTransaction 
         // new one.
 		if (is_temporary)
 		{
-            // remove old entry in the Conglomerate directory, and add new one
-			if (tempCongloms != null)
-				tempCongloms.remove(new Long(conglomId));
 			tempCongloms.put(new Long(conglomId), conglom);
 		}
 		else



Mime
View raw message