On Thu, Mar 22, 2012 at 10:20 AM, Dave Wang <[email protected]> wrote: > Is it really a requirement for HBase to have and support a synchronous > shutdown for ZK? If the answer is "yes", then we may want to change the > current non-standalone codepath to implement this. If the answer is "no", > then we should get rid of the existing code in MiniZooKeeperCluster that > implements this, which would make it easier to unify the two codepaths. >
Synchronous shutdown was probably just easier to reason around writing tests. That as best as I can remember is only the reason for it (if there ever was a 'reason'). If doing it async makes it so one code path for all deploy types and it 'works', go for it. Thanks boss, St.Ack
