hbase-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Stack <st...@duboce.net>
Subject Re: Adding support for more config options to MiniZooKeeperCluster
Date Fri, 23 Mar 2012 16:29:11 GMT
On Thu, Mar 22, 2012 at 10:20 AM, Dave Wang <dsw@cloudera.com> 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,

View raw message