From 98a5df3565beaa1999020a16fcb5338d13d5b50f Mon Sep 17 00:00:00 2001
From: Jean-Noel Rouvignac <jean-noel.rouvignac@forgerock.com>
Date: Tue, 26 Aug 2014 08:11:29 +0000
Subject: [PATCH] Removed useless getChangeNumber() and setChangeNumber methods from *Operation interfaces. These methods are useless because setChangeNumber() is never set inside production code, so changeNumber is always equal to -1 for all the update operations. In addition it is very unlikely the ChangeNumberIndexer could have computed the changeNumber before a results are sent to a persistent search.
---
opends/src/server/org/opends/server/core/DeleteOperation.java | 29 ++++++-----------------------
1 files changed, 6 insertions(+), 23 deletions(-)
diff --git a/opends/src/server/org/opends/server/core/DeleteOperation.java b/opends/src/server/org/opends/server/core/DeleteOperation.java
index 9fc6b21..a584877 100644
--- a/opends/src/server/org/opends/server/core/DeleteOperation.java
+++ b/opends/src/server/org/opends/server/core/DeleteOperation.java
@@ -22,6 +22,7 @@
*
*
* Copyright 2006-2008 Sun Microsystems, Inc.
+ * Portions copyright 2014 ForgeRock AS
*/
package org.opends.server.core;
@@ -43,7 +44,7 @@
*
* @return The raw, unprocessed entry DN as included in the client request.
*/
- public abstract ByteString getRawEntryDN();
+ ByteString getRawEntryDN();
/**
* Specifies the raw, unprocessed entry DN as included in the client request.
@@ -53,7 +54,7 @@
* @param rawEntryDN The raw, unprocessed entry DN as included in the client
* request.
*/
- public abstract void setRawEntryDN(ByteString rawEntryDN);
+ void setRawEntryDN(ByteString rawEntryDN);
/**
* Retrieves the DN of the entry to delete. This should not be called by
@@ -63,25 +64,7 @@
* @return The DN of the entry to delete, or <CODE>null</CODE> if the raw
* entry DN has not yet been processed.
*/
- public abstract DN getEntryDN();
-
- /**
- * Retrieves the change number that has been assigned to this operation.
- *
- * @return The change number that has been assigned to this operation, or -1
- * if none has been assigned yet or if there is no applicable
- * synchronization mechanism in place that uses change numbers.
- */
- public abstract long getChangeNumber();
-
- /**
- * Specifies the change number that has been assigned to this operation by the
- * synchronization mechanism.
- *
- * @param changeNumber The change number that has been assigned to this
- * operation by the synchronization mechanism.
- */
- public abstract void setChangeNumber(long changeNumber);
+ DN getEntryDN();
/**
* Retrieves the proxied authorization DN for this operation if proxied
@@ -91,7 +74,7 @@
* authorization has been requested, or {@code null} if proxied
* authorization has not been requested.
*/
- public abstract DN getProxiedAuthorizationDN();
+ DN getProxiedAuthorizationDN();
/**
* Set the proxied authorization DN for this operation if proxied
@@ -102,7 +85,7 @@
* authorization has been requested, or {@code null} if proxied
* authorization has not been requested.
*/
- public abstract void setProxiedAuthorizationDN(DN proxiedAuthorizationDN);
+ void setProxiedAuthorizationDN(DN proxiedAuthorizationDN);
}
--
Gitblit v1.10.0