This is an automated email from the ASF dual-hosted git repository.
dlmarion pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/accumulo.git
The following commit(s) were added to refs/heads/main by this push:
new fd63817409 Fixed possible flaky test issue in FateITBase (#6032)
fd63817409 is described below
commit fd63817409233a10341821210bb09721adbe27eb
Author: Dave Marion <[email protected]>
AuthorDate: Fri Dec 19 12:25:31 2025 -0500
Fixed possible flaky test issue in FateITBase (#6032)
Modified a condition in the test to account for the
case where the transaction runner thread may or may
not have started working on the fate operation.
---
test/src/main/java/org/apache/accumulo/test/fate/FateITBase.java | 8 +++++++-
1 file changed, 7 insertions(+), 1 deletion(-)
diff --git a/test/src/main/java/org/apache/accumulo/test/fate/FateITBase.java
b/test/src/main/java/org/apache/accumulo/test/fate/FateITBase.java
index c526cd6f75..278f65d33a 100644
--- a/test/src/main/java/org/apache/accumulo/test/fate/FateITBase.java
+++ b/test/src/main/java/org/apache/accumulo/test/fate/FateITBase.java
@@ -591,7 +591,13 @@ public abstract class FateITBase extends
SharedMiniClusterBase implements FateTe
fate.seedTransaction(TEST_FATE_OP, txid, new
TestRepo("testShutdownDoesNotFailTx"), true,
"Test Op");
- assertEquals(TStatus.SUBMITTED, getTxStatus(sctx, txid));
+ // The Fate operation could be in a SUBMITTED state if the
+ // Fate transaction runner thread has not picked it up yet.
+ // If the thread has picked it up, then it will be in an
+ // IN_PROGRESS state, but will be waiting on the finishCall
+ // latch to be called to continue.
+ assertTrue(TStatus.SUBMITTED == getTxStatus(sctx, txid)
+ || TStatus.IN_PROGRESS == getTxStatus(sctx, txid));
// wait for call() to be called
callStarted.await();