You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@ofbiz.apache.org by do...@apache.org on 2010/05/31 00:08:48 UTC
svn commit: r949626 -
/ofbiz/trunk/framework/entity/src/org/ofbiz/entity/jdbc/DatabaseUtil.java
Author: doogie
Date: Sun May 30 22:08:48 2010
New Revision: 949626
URL: http://svn.apache.org/viewvc?rev=949626&view=rev
Log:
Add comment that describes why there is a check of the index name existing in set.
Modified:
ofbiz/trunk/framework/entity/src/org/ofbiz/entity/jdbc/DatabaseUtil.java
Modified: ofbiz/trunk/framework/entity/src/org/ofbiz/entity/jdbc/DatabaseUtil.java
URL: http://svn.apache.org/viewvc/ofbiz/trunk/framework/entity/src/org/ofbiz/entity/jdbc/DatabaseUtil.java?rev=949626&r1=949625&r2=949626&view=diff
==============================================================================
--- ofbiz/trunk/framework/entity/src/org/ofbiz/entity/jdbc/DatabaseUtil.java (original)
+++ ofbiz/trunk/framework/entity/src/org/ofbiz/entity/jdbc/DatabaseUtil.java Sun May 30 22:08:48 2010
@@ -1583,6 +1583,8 @@ public class DatabaseUtil {
indexInfo.put(tableName, tableIndexList);
if (Debug.verboseOn()) Debug.logVerbose("Adding new Map for table: " + tableName, module);
}
+ // Need the check here, because an index can contain multiple columns,
+ // and this is represented by having multiple rows in rsCols.
if (!tableIndexList.contains(indexName)) totalIndices++;
tableIndexList.add(indexName);
} catch (SQLException e) {