This is an automated email from the ASF dual-hosted git repository.
edcoleman 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 16a3cec Fix #2078 - Marked modifier of fields store, environment and
keepRunning in Fate.java as final. (#2080)
16a3cec is described below
commit 16a3cec36981a694ecdfbefb839ecf9ddd70a535
Author: Akshit Mangotra <57808363+akshit6...@users.noreply.github.com>
AuthorDate: Wed May 5 16:02:34 2021 +0530
Fix #2078 - Marked modifier of fields store, environment and keepRunning in
Fate.java as final. (#2080)
* Fix #2078 - Marked modifier of fields store, environment and executor in
Fate.java as final.
* Fix #2078 - Reverted modifier 'executor' and marked modifier of fields
'store', 'environment' and 'keepRunning' in Fate.java as final.
---
core/src/main/java/org/apache/accumulo/fate/Fate.java | 6 +++---
1 file changed, 3 insertions(+), 3 deletions(-)
diff --git a/core/src/main/java/org/apache/accumulo/fate/Fate.java
b/core/src/main/java/org/apache/accumulo/fate/Fate.java
index a13b825..cf02c56 100644
--- a/core/src/main/java/org/apache/accumulo/fate/Fate.java
+++ b/core/src/main/java/org/apache/accumulo/fate/Fate.java
@@ -48,14 +48,14 @@ public class Fate<T> {
private static final Logger log = LoggerFactory.getLogger(Fate.class);
private final Logger runnerLog =
LoggerFactory.getLogger(TransactionRunner.class);
- private TStore<T> store;
- private T environment;
+ private final TStore<T> store;
+ private final T environment;
private ExecutorService executor;
private static final EnumSet<TStatus> FINISHED_STATES =
EnumSet.of(TStatus.FAILED, TStatus.SUCCESSFUL, TStatus.UNKNOWN);
- private AtomicBoolean keepRunning = new AtomicBoolean(true);
+ private final AtomicBoolean keepRunning = new AtomicBoolean(true);
private class TransactionRunner implements Runnable {
