You are viewing a plain text version of this content. The canonical link for it is here.
Posted to derby-commits@db.apache.org by ka...@apache.org on 2012/05/25 12:44:39 UTC
svn commit: r1342566 -
/db/derby/code/trunk/java/engine/org/apache/derby/impl/store/access/RAMTransaction.java
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