[GitHub] [accumulo] dlmarion commented on pull request #2408: Attempt to verify that MAC processes are up and running in start

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

dlmarion commented on pull request #2408:
URL: https://github.com/apache/accumulo/pull/2408#issuecomment-1005648699


   I ran all of the ITs yesterday using GitHub Actions. All succeeded except one in ExternalCompaction3_IT that timed out during a restart of the CompactionCoordinator. The test passes locally.
   
   > Another check that you may be able to use is InstanceOperations.ping()
   
   I'm wondering if I should use `InstanceOperations.waitForBalance()` instead. This would mean that the Manager and TabletServers are  up, communicating with each other, and the `root` and `metadata` tables are hosted.
   
   Also, I looked at the ITs that you mentioned. In most cases MAC is started and then these tests are stopping and starting single processes (Monitor, Manager, etc). Calling `verifyUp` after each one may be pretty costly in terms of time and may not provide much value given that the tests are currently succeeding. I think the current implementation meets the initial ask of failing if MAC fails to start at the beginning of the test.


-- 
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