From 7ac1c33413475f903e58d98bf5f0cbacbad21e07 Mon Sep 17 00:00:00 2001
From: Matthew Swift <matthew.swift@forgerock.com>
Date: Fri, 27 Feb 2015 21:36:55 +0000
Subject: [PATCH] OPENDJ-1855: Reformat and cleanup pluggable backend code: adjust visibility, findbugs, ucdetector, etc.
---
opendj-server-legacy/src/main/java/org/opends/server/backends/pluggable/DatabaseContainer.java | 12 +-
opendj-server-legacy/src/main/java/org/opends/server/backends/pluggable/Index.java | 214 +++++------------------------------
opendj-server-legacy/src/main/java/org/opends/server/backends/pluggable/NullIndex.java | 89 +++-----------
3 files changed, 59 insertions(+), 256 deletions(-)
diff --git a/opendj-server-legacy/src/main/java/org/opends/server/backends/pluggable/DatabaseContainer.java b/opendj-server-legacy/src/main/java/org/opends/server/backends/pluggable/DatabaseContainer.java
index c316a6b..5a35d0d 100644
--- a/opendj-server-legacy/src/main/java/org/opends/server/backends/pluggable/DatabaseContainer.java
+++ b/opendj-server-legacy/src/main/java/org/opends/server/backends/pluggable/DatabaseContainer.java
@@ -49,12 +49,12 @@
private static final LocalizedLogger logger = LocalizedLogger.getLoggerForThisClass();
/** The database entryContainer. */
- protected final EntryContainer entryContainer;
+ final EntryContainer entryContainer;
/** The name of the database within the entryContainer. */
- protected TreeName treeName;
+ TreeName treeName;
/** The reference to the JE Storage. */
- protected final Storage storage;
+ final Storage storage;
/**
* Create a new DatabaseContainer object.
@@ -63,7 +63,7 @@
* @param storage The JE Storage.
* @param entryContainer The entryContainer of the entry database.
*/
- protected DatabaseContainer(TreeName treeName, Storage storage, EntryContainer entryContainer)
+ DatabaseContainer(TreeName treeName, Storage storage, EntryContainer entryContainer)
{
this.storage = storage;
this.entryContainer = entryContainer;
@@ -79,7 +79,7 @@
* @throws StorageRuntimeException if a JE database error occurs while
* opening the index.
*/
- public void open(WriteableStorage txn) throws StorageRuntimeException
+ void open(WriteableStorage txn) throws StorageRuntimeException
{
txn.openTree(treeName);
if (logger.isTraceEnabled())
@@ -209,7 +209,7 @@
* @return The count of key/data pairs in the database.
* @throws StorageRuntimeException If an error occurs in the JE operation.
*/
- public long getRecordCount(ReadableStorage txn) throws StorageRuntimeException
+ long getRecordCount(ReadableStorage txn) throws StorageRuntimeException
{
long count = count(txn);
if (logger.isTraceEnabled())
diff --git a/opendj-server-legacy/src/main/java/org/opends/server/backends/pluggable/Index.java b/opendj-server-legacy/src/main/java/org/opends/server/backends/pluggable/Index.java
index a861cda..95166c1 100644
--- a/opendj-server-legacy/src/main/java/org/opends/server/backends/pluggable/Index.java
+++ b/opendj-server-legacy/src/main/java/org/opends/server/backends/pluggable/Index.java
@@ -46,7 +46,6 @@
import org.opends.server.backends.pluggable.spi.StorageRuntimeException;
import org.opends.server.backends.pluggable.spi.TreeName;
import org.opends.server.backends.pluggable.spi.WriteableStorage;
-import org.opends.server.types.DirectoryException;
import org.opends.server.types.Entry;
import org.opends.server.types.Modification;
import org.opends.server.util.StaticUtils;
@@ -146,13 +145,6 @@
}
}
- /**
- * Add an add entry ID operation into a index buffer.
- *
- * @param buffer The index buffer to insert the ID into.
- * @param keyBytes The index key bytes.
- * @param entryID The entry ID.
- */
final void insertID(IndexBuffer buffer, ByteString keyBytes, EntryID entryID)
{
getBufferedIndexValues(buffer, keyBytes).addEntryID(keyBytes, entryID);
@@ -215,15 +207,6 @@
put(txn, key, value);
}
- /**
- * Update the set of entry IDs for a given key.
- *
- * @param txn A database transaction, or null if none is required.
- * @param key The database key.
- * @param deletedIDs The IDs to remove for the key.
- * @param addedIDs the IDs to add for the key.
- * @throws StorageRuntimeException If a database error occurs.
- */
void updateKey(WriteableStorage txn, ByteString key, EntryIDSet deletedIDs, EntryIDSet addedIDs)
throws StorageRuntimeException
{
@@ -287,14 +270,11 @@
return entryIDSet != null && entryIDSet.size() > 0;
}
- private void updateKeyWithRMW(WriteableStorage txn,
- ByteString key,
- EntryIDSet deletedIDs,
- EntryIDSet addedIDs)
+ private void updateKeyWithRMW(WriteableStorage txn, ByteString key, EntryIDSet deletedIDs, EntryIDSet addedIDs)
throws StorageRuntimeException
{
final ByteString value = read(txn, key, true);
- if(value != null)
+ if (value != null)
{
EntryIDSet entryIDList = computeEntryIDList(key, value, deletedIDs, addedIDs);
ByteString after = entryIDList.toByteString();
@@ -384,13 +364,6 @@
return entryIDList;
}
- /**
- * Add an remove entry ID operation into a index buffer.
- *
- * @param buffer The index buffer to insert the ID into.
- * @param keyBytes The index key bytes.
- * @param entryID The entry ID.
- */
final void removeID(IndexBuffer buffer, ByteString keyBytes, EntryID entryID)
{
getBufferedIndexValues(buffer, keyBytes).deleteEntryID(keyBytes, entryID);
@@ -409,12 +382,7 @@
logger.error(ERR_JEB_INDEX_CORRUPT_REQUIRES_REBUILD, treeName);
}
- /**
- * Buffered delete of a key from the JE database.
- * @param buffer The index buffer to use to store the deleted keys
- * @param keyBytes The index key bytes.
- */
- public void delete(IndexBuffer buffer, ByteString keyBytes)
+ void delete(IndexBuffer buffer, ByteString keyBytes)
{
getBufferedIndexValues(buffer, keyBytes);
}
@@ -427,15 +395,18 @@
/**
* Check if an entry ID is in the set of IDs indexed by a given key.
*
- * @param txn A database transaction, or null if none is required.
- * @param key The index key.
- * @param entryID The entry ID.
- * @return true if the entry ID is indexed by the given key,
- * false if it is not indexed by the given key,
- * undefined if the key has exceeded the entry limit.
- * @throws StorageRuntimeException If an error occurs in the JE database.
+ * @param txn
+ * A database transaction.
+ * @param key
+ * The index key.
+ * @param entryID
+ * The entry ID.
+ * @return true if the entry ID is indexed by the given key, false if it is not indexed by the
+ * given key, undefined if the key has exceeded the entry limit.
+ * @throws StorageRuntimeException
+ * If an error occurs in the JE database.
*/
- public ConditionResult containsID(ReadableStorage txn, ByteString key, EntryID entryID)
+ ConditionResult containsID(ReadableStorage txn, ByteString key, EntryID entryID)
throws StorageRuntimeException
{
if(rebuildRunning)
@@ -463,14 +434,7 @@
}
}
- /**
- * Reads the set of entry IDs for a given key.
- *
- * @param key The database key.
- * @param txn A database transaction, or null if none is required.
- * @return The entry IDs indexed by this key.
- */
- public EntryIDSet readKey(ByteSequence key, ReadableStorage txn)
+ EntryIDSet readKey(ByteSequence key, ReadableStorage txn)
{
if(rebuildRunning)
{
@@ -501,33 +465,6 @@
}
/**
- * Writes the set of entry IDs for a given key.
- *
- * @param key The database key.
- * @param entryIDList The entry IDs indexed by this key.
- * @param txn A database transaction, or null if none is required.
- * @throws StorageRuntimeException If an error occurs in the JE database.
- */
- public void writeKey(WriteableStorage txn, ByteString key, EntryIDSet entryIDList)
- throws StorageRuntimeException
- {
- ByteString value = entryIDList.toByteString();
- if (!value.isEmpty())
- {
- if (!entryIDList.isDefined())
- {
- entryLimitExceededCount++;
- }
- put(txn, key, value);
- }
- else
- {
- // No more IDs, so remove the key.
- delete(txn, key);
- }
- }
-
- /**
* Reads a range of keys and collects all their entry IDs into a
* single set.
*
@@ -550,7 +487,7 @@
* specified.
* @return The set of entry IDs.
*/
- public EntryIDSet readRange(ReadableStorage txn,
+ EntryIDSet readRange(ReadableStorage txn,
ByteSequence lower, ByteSequence upper, boolean lowerIncluded, boolean upperIncluded)
{
// If this index is not trusted, then just return an undefined id set.
@@ -637,29 +574,13 @@
}
}
- /**
- * Get the number of keys that have exceeded the entry limit since this
- * object was created.
- * @return The number of keys that have exceeded the entry limit since this
- * object was created.
- */
- public int getEntryLimitExceededCount()
+ int getEntryLimitExceededCount()
{
return entryLimitExceededCount;
}
- /**
- * Update the index buffer for a deleted entry.
- *
- * @param buffer The index buffer to use to store the deleted keys
- * @param entryID The entry ID.
- * @param entry The entry to be indexed.
- * @param options The indexing options to use
- * @throws StorageRuntimeException If an error occurs in the JE database.
- * @throws DirectoryException If a Directory Server error occurs.
- */
- public void addEntry(IndexBuffer buffer, EntryID entryID, Entry entry,
- IndexingOptions options) throws StorageRuntimeException, DirectoryException
+ void addEntry(IndexBuffer buffer, EntryID entryID, Entry entry, IndexingOptions options)
+ throws StorageRuntimeException
{
HashSet<ByteString> addKeys = new HashSet<ByteString>();
indexer.indexEntry(entry, addKeys, options);
@@ -670,18 +591,8 @@
}
}
- /**
- * Update the index buffer for a deleted entry.
- *
- * @param buffer The index buffer to use to store the deleted keys
- * @param entryID The entry ID
- * @param entry The contents of the deleted entry.
- * @param options The indexing options to use
- * @throws StorageRuntimeException If an error occurs in the JE database.
- * @throws DirectoryException If a Directory Server error occurs.
- */
- public void removeEntry(IndexBuffer buffer, EntryID entryID, Entry entry,
- IndexingOptions options) throws StorageRuntimeException, DirectoryException
+ void removeEntry(IndexBuffer buffer, EntryID entryID, Entry entry, IndexingOptions options)
+ throws StorageRuntimeException
{
HashSet<ByteString> delKeys = new HashSet<ByteString>();
indexer.indexEntry(entry, delKeys, options);
@@ -692,24 +603,8 @@
}
}
- /**
- * Update the index to reflect a sequence of modifications in a Modify
- * operation.
- *
- * @param buffer The index buffer to use to store the deleted keys
- * @param entryID The ID of the entry that was modified.
- * @param oldEntry The entry before the modifications were applied.
- * @param newEntry The entry after the modifications were applied.
- * @param mods The sequence of modifications in the Modify operation.
- * @param options The indexing options to use
- * @throws StorageRuntimeException If an error occurs in the JE database.
- */
- public void modifyEntry(IndexBuffer buffer,
- EntryID entryID,
- Entry oldEntry,
- Entry newEntry,
- List<Modification> mods, IndexingOptions options)
- throws StorageRuntimeException
+ void modifyEntry(IndexBuffer buffer, EntryID entryID, Entry oldEntry, Entry newEntry, List<Modification> mods,
+ IndexingOptions options) throws StorageRuntimeException
{
TreeMap<ByteString, Boolean> modifiedKeys = new TreeMap<ByteString, Boolean>();
indexer.modifyEntry(oldEntry, newEntry, mods, modifiedKeys, options);
@@ -727,88 +622,45 @@
}
}
- /**
- * Set the index entry limit.
- *
- * @param indexEntryLimit The index entry limit to set.
- * @return True if a rebuild is required or false otherwise.
- */
- public boolean setIndexEntryLimit(int indexEntryLimit)
+ boolean setIndexEntryLimit(int indexEntryLimit)
{
- final boolean rebuildRequired =
- this.indexEntryLimit < indexEntryLimit && entryLimitExceededCount > 0;
+ final boolean rebuildRequired = this.indexEntryLimit < indexEntryLimit && entryLimitExceededCount > 0;
this.indexEntryLimit = indexEntryLimit;
return rebuildRequired;
}
- /**
- * Set the indexer.
- *
- * @param indexer The indexer to set
- */
final void setIndexer(Indexer indexer)
{
this.indexer = indexer;
}
- /**
- * Return entry limit.
- *
- * @return The entry limit.
- */
- public int getIndexEntryLimit() {
- return this.indexEntryLimit;
+ int getIndexEntryLimit()
+ {
+ return indexEntryLimit;
}
- /**
- * Set the index trust state.
- * @param txn A database transaction, or null if none is required.
- * @param trusted True if this index should be trusted or false
- * otherwise.
- * @throws StorageRuntimeException If an error occurs in the JE database.
- */
- public synchronized void setTrusted(WriteableStorage txn, boolean trusted)
- throws StorageRuntimeException
+ synchronized void setTrusted(WriteableStorage txn, boolean trusted) throws StorageRuntimeException
{
this.trusted = trusted;
state.putIndexTrustState(txn, this, trusted);
}
- /**
- * Return true iff this index is trusted.
- * @return the trusted state of this index
- */
- public synchronized boolean isTrusted()
+ synchronized boolean isTrusted()
{
return trusted;
}
- /**
- * Return <code>true</code> iff this index is being rebuilt.
- * @return The rebuild state of this index
- */
- public synchronized boolean isRebuildRunning()
+ synchronized boolean isRebuildRunning()
{
return rebuildRunning;
}
- /**
- * Set the rebuild status of this index.
- * @param rebuildRunning True if a rebuild process on this index
- * is running or False otherwise.
- */
- public synchronized void setRebuildStatus(boolean rebuildRunning)
+ synchronized void setRebuildStatus(boolean rebuildRunning)
{
this.rebuildRunning = rebuildRunning;
}
- /**
- * Whether this index maintains a count of IDs for keys once the
- * entry limit has exceeded.
- * @return <code>true</code> if this index maintains court of IDs
- * or <code>false</code> otherwise
- */
- public boolean getMaintainCount()
+ boolean getMaintainCount()
{
return maintainCount;
}
diff --git a/opendj-server-legacy/src/main/java/org/opends/server/backends/pluggable/NullIndex.java b/opendj-server-legacy/src/main/java/org/opends/server/backends/pluggable/NullIndex.java
index 536d6f4..8e5c8b7 100644
--- a/opendj-server-legacy/src/main/java/org/opends/server/backends/pluggable/NullIndex.java
+++ b/opendj-server-legacy/src/main/java/org/opends/server/backends/pluggable/NullIndex.java
@@ -35,7 +35,6 @@
import org.opends.server.backends.pluggable.spi.StorageRuntimeException;
import org.opends.server.backends.pluggable.spi.TreeName;
import org.opends.server.backends.pluggable.spi.WriteableStorage;
-import org.opends.server.types.DirectoryException;
import org.opends.server.types.Entry;
import org.opends.server.types.Modification;
@@ -46,31 +45,12 @@
final class NullIndex extends Index
{
- /**
- * Create a new null index object.
- *
- * @param name
- * The name of the index database within the entryContainer.
- * @param indexer
- * The indexer object to construct index keys from LDAP attribute values.
- * @param state
- * The state database to persist index state info.
- * @param storage
- * The JE Storage
- * @param txn
- * The transaction to use when creating this object
- * @param entryContainer
- * The database entryContainer holding this index.
- * @throws StorageRuntimeException
- * If an error occurs in the JE database.
- */
- public NullIndex(TreeName name, Indexer indexer, State state, Storage storage, WriteableStorage txn,
+ NullIndex(TreeName name, Indexer indexer, State state, Storage storage, WriteableStorage txn,
EntryContainer entryContainer) throws StorageRuntimeException
{
super(name, indexer, state, 0, 0, false, storage, txn, entryContainer);
}
- /** {@inheritDoc} */
@Override
void updateKey(WriteableStorage txn, ByteString key, EntryIDSet deletedIDs, EntryIDSet addedIDs)
throws StorageRuntimeException
@@ -78,167 +58,138 @@
// Do nothing.
}
- /** {@inheritDoc} */
@Override
- public void delete(IndexBuffer buffer, ByteString keyBytes)
+ void delete(IndexBuffer buffer, ByteString keyBytes)
{
// Do nothing.
}
- /** {@inheritDoc} */
@Override
- public ConditionResult containsID(ReadableStorage txn, ByteString key, EntryID entryID)
+ ConditionResult containsID(ReadableStorage txn, ByteString key, EntryID entryID)
throws StorageRuntimeException
{
return ConditionResult.UNDEFINED;
}
- /** {@inheritDoc} */
@Override
- public EntryIDSet readKey(ByteSequence key, ReadableStorage txn)
+ EntryIDSet readKey(ByteSequence key, ReadableStorage txn)
{
return new EntryIDSet();
}
- /** {@inheritDoc} */
@Override
- public void writeKey(WriteableStorage txn, ByteString key, EntryIDSet entryIDList) throws StorageRuntimeException
- {
- // Do nothing.
- }
-
- /** {@inheritDoc} */
- @Override
- public EntryIDSet readRange(ReadableStorage txn, ByteSequence lower, ByteSequence upper, boolean lowerIncluded,
+ EntryIDSet readRange(ReadableStorage txn, ByteSequence lower, ByteSequence upper, boolean lowerIncluded,
boolean upperIncluded)
{
return new EntryIDSet();
}
- /** {@inheritDoc} */
@Override
- public int getEntryLimitExceededCount()
+ int getEntryLimitExceededCount()
{
return 0;
}
- /** {@inheritDoc} */
@Override
- public void addEntry(IndexBuffer buffer, EntryID entryID, Entry entry, IndexingOptions options)
- throws StorageRuntimeException, DirectoryException
+ void addEntry(IndexBuffer buffer, EntryID entryID, Entry entry, IndexingOptions options)
+ throws StorageRuntimeException
{
// Do nothing.
}
- /** {@inheritDoc} */
@Override
- public void removeEntry(IndexBuffer buffer, EntryID entryID, Entry entry, IndexingOptions options)
- throws StorageRuntimeException, DirectoryException
+ void removeEntry(IndexBuffer buffer, EntryID entryID, Entry entry, IndexingOptions options)
+ throws StorageRuntimeException
{
// Do nothing.
}
- /** {@inheritDoc} */
@Override
- public void modifyEntry(IndexBuffer buffer, EntryID entryID, Entry oldEntry, Entry newEntry, List<Modification> mods,
+ void modifyEntry(IndexBuffer buffer, EntryID entryID, Entry oldEntry, Entry newEntry, List<Modification> mods,
IndexingOptions options) throws StorageRuntimeException
{
// Do nothing.
}
- /** {@inheritDoc} */
@Override
- public boolean setIndexEntryLimit(int indexEntryLimit)
+ boolean setIndexEntryLimit(int indexEntryLimit)
{
return false;
}
- /** {@inheritDoc} */
@Override
- public int getIndexEntryLimit()
+ int getIndexEntryLimit()
{
return 0;
}
- /** {@inheritDoc} */
@Override
- public void setTrusted(WriteableStorage txn, boolean trusted) throws StorageRuntimeException
+ void setTrusted(WriteableStorage txn, boolean trusted) throws StorageRuntimeException
{
// Do nothing.
}
- /** {@inheritDoc} */
@Override
- public boolean isTrusted()
+ boolean isTrusted()
{
return true;
}
- /** {@inheritDoc} */
@Override
- public boolean isRebuildRunning()
+ boolean isRebuildRunning()
{
return false;
}
- /** {@inheritDoc} */
@Override
- public void setRebuildStatus(boolean rebuildRunning)
+ void setRebuildStatus(boolean rebuildRunning)
{
// Do nothing.
}
- /** {@inheritDoc} */
@Override
- public boolean getMaintainCount()
+ boolean getMaintainCount()
{
return false;
}
- /** {@inheritDoc} */
@Override
- public void open(WriteableStorage txn) throws StorageRuntimeException
+ void open(WriteableStorage txn) throws StorageRuntimeException
{
// Do nothing.
}
- /** {@inheritDoc} */
@Override
public void close() throws StorageRuntimeException
{
// Do nothing.
}
- /** {@inheritDoc} */
@Override
void put(WriteableStorage txn, ByteSequence key, ByteSequence value) throws StorageRuntimeException
{
}
- /** {@inheritDoc} */
@Override
ByteString read(ReadableStorage txn, ByteSequence key, boolean isRMW) throws StorageRuntimeException
{
return null;
}
- /** {@inheritDoc} */
@Override
boolean insert(WriteableStorage txn, ByteString key, ByteString value) throws StorageRuntimeException
{
return true;
}
- /** {@inheritDoc} */
@Override
boolean delete(WriteableStorage txn, ByteSequence key) throws StorageRuntimeException
{
return true;
}
- /** {@inheritDoc} */
@Override
- public long getRecordCount(ReadableStorage txn) throws StorageRuntimeException
+ long getRecordCount(ReadableStorage txn) throws StorageRuntimeException
{
return 0;
}
--
Gitblit v1.10.0