diff options
Diffstat (limited to 'gerrit-server/src/main/java/com/google/gerrit/server/update/BatchUpdateOp.java')
| -rw-r--r-- | gerrit-server/src/main/java/com/google/gerrit/server/update/BatchUpdateOp.java | 7 |
1 files changed, 5 insertions, 2 deletions
diff --git a/gerrit-server/src/main/java/com/google/gerrit/server/update/BatchUpdateOp.java b/gerrit-server/src/main/java/com/google/gerrit/server/update/BatchUpdateOp.java index 39e25dda74..87a43a354c 100644 --- a/gerrit-server/src/main/java/com/google/gerrit/server/update/BatchUpdateOp.java +++ b/gerrit-server/src/main/java/com/google/gerrit/server/update/BatchUpdateOp.java @@ -22,8 +22,11 @@ package com.google.gerrit.server.update; * BatchUpdate#addOp(com.google.gerrit.reviewdb.client.Change.Id, BatchUpdateOp)}. * * <p>Usually, a single {@code BatchUpdateOp} instance is only associated with a single change, i.e. - * {@code addOp} is only called once with that instance. This allows an instance to communicate - * between phases by storing data in private fields. + * {@code addOp} is only called once with that instance. Additionally, each method in {@code + * BatchUpdateOp} is called at most once per {@link BatchUpdate} execution. + * + * <p>Taken together, these two properties mean an instance may communicate between phases by + * storing data in private fields, and a single instance must not be reused. */ public interface BatchUpdateOp extends RepoOnlyOp { /** |