diff --git a/gerrit-server/src/main/java/com/google/gerrit/server/extensions/events/ChangeIndexedListener.java b/gerrit-extension-api/src/main/java/com/google/gerrit/extensions/events/ChangeIndexedListener.java similarity index 79% rename from gerrit-server/src/main/java/com/google/gerrit/server/extensions/events/ChangeIndexedListener.java rename to gerrit-extension-api/src/main/java/com/google/gerrit/extensions/events/ChangeIndexedListener.java index f996724ec7..fd8dac87ef 100644 --- a/gerrit-server/src/main/java/com/google/gerrit/server/extensions/events/ChangeIndexedListener.java +++ b/gerrit-extension-api/src/main/java/com/google/gerrit/extensions/events/ChangeIndexedListener.java @@ -12,18 +12,16 @@ // See the License for the specific language governing permissions and // limitations under the License. -package com.google.gerrit.server.extensions.events; +package com.google.gerrit.extensions.events; import com.google.gerrit.extensions.annotations.ExtensionPoint; -import com.google.gerrit.reviewdb.client.Change; -import com.google.gerrit.server.query.change.ChangeData; /** Notified whenever a change is indexed or deleted from the index. */ @ExtensionPoint public interface ChangeIndexedListener { /** Invoked when a change is indexed. */ - void onChangeIndexed(ChangeData change); + void onChangeIndexed(int id); /** Invoked when a change is deleted from the index. */ - void onChangeDeleted(Change.Id id); + void onChangeDeleted(int id); } diff --git a/gerrit-server/src/main/java/com/google/gerrit/server/config/GerritGlobalModule.java b/gerrit-server/src/main/java/com/google/gerrit/server/config/GerritGlobalModule.java index 9781f39de5..6bbfd6284a 100644 --- a/gerrit-server/src/main/java/com/google/gerrit/server/config/GerritGlobalModule.java +++ b/gerrit-server/src/main/java/com/google/gerrit/server/config/GerritGlobalModule.java @@ -31,6 +31,7 @@ import com.google.gerrit.extensions.config.ExternalIncludedIn; import com.google.gerrit.extensions.config.FactoryModule; import com.google.gerrit.extensions.events.AgreementSignupListener; import com.google.gerrit.extensions.events.ChangeAbandonedListener; +import com.google.gerrit.extensions.events.ChangeIndexedListener; import com.google.gerrit.extensions.events.ChangeMergedListener; import com.google.gerrit.extensions.events.ChangeRestoredListener; import com.google.gerrit.extensions.events.CommentAddedListener; @@ -97,7 +98,6 @@ import com.google.gerrit.server.change.ChangeKindCacheImpl; import com.google.gerrit.server.change.MergeabilityCacheImpl; import com.google.gerrit.server.events.EventFactory; import com.google.gerrit.server.events.EventsMetrics; -import com.google.gerrit.server.extensions.events.ChangeIndexedListener; import com.google.gerrit.server.extensions.events.GitReferenceUpdated; import com.google.gerrit.server.git.BatchUpdate; import com.google.gerrit.server.git.EmailMerge; diff --git a/gerrit-server/src/main/java/com/google/gerrit/server/index/change/ChangeIndexer.java b/gerrit-server/src/main/java/com/google/gerrit/server/index/change/ChangeIndexer.java index 393f6844ba..4a331d9c75 100644 --- a/gerrit-server/src/main/java/com/google/gerrit/server/index/change/ChangeIndexer.java +++ b/gerrit-server/src/main/java/com/google/gerrit/server/index/change/ChangeIndexer.java @@ -20,12 +20,12 @@ import com.google.common.util.concurrent.CheckedFuture; import com.google.common.util.concurrent.Futures; import com.google.common.util.concurrent.ListenableFuture; import com.google.common.util.concurrent.ListeningExecutorService; +import com.google.gerrit.extensions.events.ChangeIndexedListener; import com.google.gerrit.extensions.registration.DynamicSet; import com.google.gerrit.reviewdb.client.Change; import com.google.gerrit.reviewdb.client.Project; import com.google.gerrit.reviewdb.server.ReviewDb; import com.google.gerrit.server.CurrentUser; -import com.google.gerrit.server.extensions.events.ChangeIndexedListener; import com.google.gerrit.server.index.Index; import com.google.gerrit.server.notedb.ChangeNotes; import com.google.gerrit.server.notedb.NotesMigration; @@ -180,16 +180,16 @@ public class ChangeIndexer { for (Index i : getWriteIndexes()) { i.replace(cd); } - fireChangeIndexedEvent(cd); + fireChangeIndexedEvent(cd.getId().get()); } - private void fireChangeIndexedEvent(ChangeData change) { + private void fireChangeIndexedEvent(int id) { for (ChangeIndexedListener listener : indexedListener) { - listener.onChangeIndexed(change); + listener.onChangeIndexed(id); } } - private void fireChangeDeletedFromIndexEvent(Change.Id id) { + private void fireChangeDeletedFromIndexEvent(int id) { for (ChangeIndexedListener listener : indexedListener) { listener.onChangeDeleted(id); } @@ -338,7 +338,7 @@ public class ChangeIndexer { for (ChangeIndex i : getWriteIndexes()) { i.delete(id); } - fireChangeDeletedFromIndexEvent(id); + fireChangeDeletedFromIndexEvent(id.get()); return null; } }