ctubbsii commented on issue #2474:
URL: https://github.com/apache/accumulo/issues/2474#issuecomment-1060713495


   I'm pretty sure we'd have to do it before each call to get the status, since 
the status is updated in a separate thread by a different ZK client. However, 
we don't need to have all the lines I had. I was told that the path parameter 
wasn't really used, but from my experimentation, it seemed that the path did 
need to exist at least. So, using one sync call to the "/" path should suffice 
inside the get status method.
   
   > I could be mis-remembering - but fateTransactionStatus is for metrics. 
(added 1.10?)
   
   It might be used for that, but in this test, it's being used to check for 
expected progress through the FaTE lifecycle.


-- 
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: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]


Reply via email to