[GitHub] [accumulo] ctubbsii commented on a change in pull request #2492: Add goal message to Fate. Closes #1249

[GitBox <gi...@apache.org>]

ctubbsii commented on a change in pull request #2492:
URL: https://github.com/apache/accumulo/pull/2492#discussion_r806033913



##########
File path: core/src/main/java/org/apache/accumulo/fate/Fate.java
##########
@@ -268,18 +268,18 @@ public long startTransaction() {
 
   // start work in the transaction.. it is safe to call this
   // multiple times for a transaction... but it will only seed once
-  public void seedTransaction(long tid, Repo<T> repo, boolean autoCleanUp) {
+  public void seedTransaction(long tid, Repo<T> repo, boolean autoCleanUp, String goalMessage) {
     store.reserve(tid);
     try {
       if (store.getStatus(tid) == TStatus.NEW) {
         if (store.top(tid) == null) {
           try {
+            log.info("FATE TxId: {} {}", tid, goalMessage);

Review comment:
       Should these be debug instead of info? Info seems like it might be a bit noisy.




-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: notifications-unsubscribe@accumulo.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org