This looks similar to the Coproc in TestReplicasClient. Maybe there's some reuse between them, or a shared library of coprocs for tests?
It's totally similar
But it's always difficult to share code between tests.
nit: Should just use the constant
Better to add some kind of max iteration count here to protect our poor jenkins box from itself.
Agreed. I added Waiter.waitFor
disableTable and enableTable are both synchronous, but modifyTable is asynchronous
I'm not sure that modifyTable is really asynchronous (despite the doc): we release the lock synchronously, so I expect it will work. I think that we have an issue if it does not.
v3 on his way.