mirror of https://github.com/OpenIdentityPlatform/OpenDJ.git

Jean-Noel Rouvignac
13.45.2014 f0fd842eb268fb222a85d68ac68ba8e564054bcc
OPENDJ-1591 (CR-5256) Switch to SDK matching rules

Only the byte array comparator needs to be serializable (because JE btree comparators must be serializable).
ByteSequence comparator has no such need, so the intermediate AtributeIndex.BSKeyComparator is not needed anymore.

AtributeIndex.java:
Removed BSKeyComparator inner class and directly used ByteSequence.COMPARATOR instead.
2 files modified
23 ■■■■■ changed files
opendj-sdk/opendj3-server-dev/src/server/org/opends/server/backends/jeb/AttributeIndex.java 20 ●●●●● patch | view | raw | blame | history
opendj-sdk/opendj3-server-dev/src/server/org/opends/server/backends/jeb/Indexer.java 3 ●●●● patch | view | raw | blame | history
opendj-sdk/opendj3-server-dev/src/server/org/opends/server/backends/jeb/AttributeIndex.java
@@ -707,26 +707,6 @@
  }
  /**
   * Delegator to {@link ByteSequence#COMPARATOR}.
   * <p>
   * This intermediate class is necessary to satisfy JE's requirements for a btree comparator.
   *
   * @see com.sleepycat.je.DatabaseConfig#setBtreeComparator(Comparator)
   */
  public static class BSKeyComparator implements Comparator<ByteSequence>
  {
    /** The instance. */
    public static final BSKeyComparator INSTANCE = new BSKeyComparator();
    /** {@inheritDoc} */
    @Override
    public int compare(ByteSequence a, ByteSequence b)
    {
      return ByteSequence.COMPARATOR.compare(a, b);
    }
  }
  /**
   * Close cursors related to the attribute indexes.
   *
   * @throws DatabaseException If a database error occurs.
opendj-sdk/opendj3-server-dev/src/server/org/opends/server/backends/jeb/Indexer.java
@@ -34,7 +34,6 @@
import org.forgerock.opendj.ldap.ByteSequence;
import org.forgerock.opendj.ldap.ByteString;
import org.forgerock.opendj.ldap.spi.IndexingOptions;
import org.opends.server.backends.jeb.AttributeIndex.BSKeyComparator;
import org.opends.server.backends.jeb.AttributeIndex.KeyComparator;
import org.opends.server.types.Entry;
import org.opends.server.types.Modification;
@@ -64,7 +63,7 @@
   */
  public final Comparator<ByteSequence> getBSComparator()
  {
    return BSKeyComparator.INSTANCE;
    return ByteSequence.COMPARATOR;
  }
  /**