You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@directory.apache.org by go...@apache.org on 2012/08/11 07:39:42 UTC
svn commit: r1371870 -
/directory/apacheds/branches/apacheds-txns/core-shared/src/main/java/org/apache/directory/server/core/shared/partition/DefaultOperationExecutionManager.java
Author: gokturk
Date: Sat Aug 11 05:39:42 2012
New Revision: 1371870
URL: http://svn.apache.org/viewvc?rev=1371870&view=rev
Log:
* Fix for objectclass index to don't contain "top" as a key which will bloat the index unnecessarly since it's present on every entry. This was documented but didn't applied in the code.(Merged from trunk)
Modified:
directory/apacheds/branches/apacheds-txns/core-shared/src/main/java/org/apache/directory/server/core/shared/partition/DefaultOperationExecutionManager.java
Modified: directory/apacheds/branches/apacheds-txns/core-shared/src/main/java/org/apache/directory/server/core/shared/partition/DefaultOperationExecutionManager.java
URL: http://svn.apache.org/viewvc/directory/apacheds/branches/apacheds-txns/core-shared/src/main/java/org/apache/directory/server/core/shared/partition/DefaultOperationExecutionManager.java?rev=1371870&r1=1371869&r2=1371870&view=diff
==============================================================================
--- directory/apacheds/branches/apacheds-txns/core-shared/src/main/java/org/apache/directory/server/core/shared/partition/DefaultOperationExecutionManager.java (original)
+++ directory/apacheds/branches/apacheds-txns/core-shared/src/main/java/org/apache/directory/server/core/shared/partition/DefaultOperationExecutionManager.java Sat Aug 11 05:39:42 2012
@@ -174,6 +174,11 @@ public class DefaultOperationExecutionMa
for ( Value<?> value : objectClass )
{
+ if ( value.equals( SchemaConstants.TOP_OC ) )
+ {
+ continue;
+ }
+
indexChange = new IndexChange( objectClassIdx, value.getString(),
id, IndexChange.Type.ADD, true );
changeContainer.addChange( indexChange );
@@ -504,7 +509,7 @@ public class DefaultOperationExecutionMa
}
catch ( Exception e )
{
- e.printStackTrace();
+ e.printStackTrace();
throw new LdapOperationErrorException( e.getMessage(), e );
}
}
@@ -708,6 +713,11 @@ public class DefaultOperationExecutionMa
for ( Value<?> value : mods )
{
+ if ( value.equals( SchemaConstants.TOP_OC ) )
+ {
+ continue;
+ }
+
indexChange = new IndexChange( objectClassIdx, value.getString(), id, IndexChange.Type.ADD,
true );
changeContainer.addChange( indexChange );