commit 815133ab15d20f9af5e36f58438ee28d916880e2
author Martin Fick <mfick@codeaurora.org> Thu Feb 13 17:00:58 2020 -0700
committer Sumit LNU <suchawla@codeaurora.org> Mon Jun 07 10:24:41 2021 +0530
tree f9df00cf39f4a5d999d767e1a444f5e06b9a928a
parent 8019d4141f6b4e4318272c49c6aaebbaccdc69b1

fixup! Fix event plugin possible transaction reuse bug

Fix the paths that were not updated when adding the 'next' directory.

Change-Id: I34e084f8d00bc9f0017c33678ba817720f769565

src/main/java/com/googlesource/gerrit/plugins/events/fsstore/UpdatableFileValue.java

diff --git a/src/main/java/com/googlesource/gerrit/plugins/events/fsstore/UpdatableFileValue.java b/src/main/java/com/googlesource/gerrit/plugins/events/fsstore/UpdatableFileValue.java
index 59cb13e..9e8f370 100644
--- a/src/main/java/com/googlesource/gerrit/plugins/events/fsstore/UpdatableFileValue.java
+++ b/src/main/java/com/googlesource/gerrit/plugins/events/fsstore/UpdatableFileValue.java
@@ -211,9 +211,9 @@
         // when closed (operations depend on "closed" in paths).
         perserve();
 
-        // rename update/<uuid>/closed/value(next) -> value
+        // rename update/<uuid>/next/closed/value(next) -> value
         committed = myCommit = Fs.tryAtomicMove(upaths.value, updatable.path); // Phase 5
-        // now there should be: update/<uuid>/closed/ and: value (file)
+        // now there should be: update/<uuid>/next/closed/ and: value (file)
       }
       if (!committed && closed) {
         committed = !Files.exists(upaths.value);