Do not specify the comparator and rely on ByteString natural ordering.

Jean-Noel Rouvignac

23.40.2014 c91fb5a1fc804b6b3943cfc5c2534cd54307c749

Do not specify the comparator and rely on ByteString natural ordering.

 opendj3-server-dev/src/server/org/opends/server/backends/pluggable/Index.java

@@ -26,6 +26,8 @@
 */
package org.opends.server.backends.pluggable;

import static org.opends.messages.JebMessages.*;

import java.util.ArrayList;
import java.util.HashSet;
import java.util.List;
@@ -49,8 +51,6 @@
import org.opends.server.types.Modification;
import org.opends.server.util.StaticUtils;

import static org.opends.messages.JebMessages.*;

/**
 * Represents an index implemented by a JE database in which each key maps to
 * a set of entry IDs.  The key is a byte array, and is constructed from some
@@ -666,8 +666,7 @@
                          List<Modification> mods, IndexingOptions options)
      throws StorageRuntimeException
  {
    TreeMap<ByteString, Boolean> modifiedKeys =
        new TreeMap<ByteString, Boolean>(ByteSequence.COMPARATOR);
    TreeMap<ByteString, Boolean> modifiedKeys = new TreeMap<ByteString, Boolean>();
    indexer.modifyEntry(oldEntry, newEntry, mods, modifiedKeys, options);

    for (Map.Entry<ByteString, Boolean> modifiedKey : modifiedKeys.entrySet())

 opendj3-server-dev/src/server/org/opends/server/backends/pluggable/IndexBuffer.java

@@ -32,7 +32,6 @@
import java.util.TreeMap;
import java.util.TreeSet;

import org.forgerock.opendj.ldap.ByteSequence;
import org.forgerock.opendj.ldap.ByteString;
import org.opends.server.backends.pluggable.spi.StorageRuntimeException;
import org.opends.server.backends.pluggable.spi.WriteableStorage;
@@ -209,7 +208,7 @@
    TreeMap<ByteString, BufferedIndexValues> bufferedOperations = bufferedIndexes.get(index);
    if (bufferedOperations == null)
    {
      bufferedOperations = new TreeMap<ByteString, BufferedIndexValues>(ByteSequence.COMPARATOR);
      bufferedOperations = new TreeMap<ByteString, BufferedIndexValues>();
      bufferedIndexes.put(index, bufferedOperations);
    }
    else