hbase-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Dave Wang <...@cloudera.com>
Subject Re: Adding support for more config options to MiniZooKeeperCluster
Date Thu, 22 Mar 2012 17:20:48 GMT
Stack,

I took a first whack at this.

It's going to be impossible to unify the two configure/start codepaths
exactly, as the standalone HBase path currently requires a synchronous
shutdown() path for the MiniZooKeeperCluster, whereas the non-standalone
path doesn't (it currently just relies on the scripts to kill the zookeeper
processes).  In fact, the non-standalone codepath doesn't have a good way
of having a synchronous shutdown path without some changes.

I noticed that a bunch of unit tests have synchronous shutdown calls, which
is perhaps not surprising considering MiniZooKeeperCluster came from unit
test code.  But this all seems limited to only the standalone HBase case,
and I don't see why this has to be different for the standalone case.

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.

- Dave

On Tue, Mar 20, 2012 at 11:41 AM, Stack <stack@duboce.net> wrote:

> On Mon, Mar 19, 2012 at 3:39 PM, Dave Wang <dsw@cloudera.com> wrote:
> > I wanted to add tickTime and min/max timeout support into
> > MiniZooKeeperCluster, and have a small patch to do that ready to go.
> >  However, I was wondering if it makes more sense to make the codepaths
> for
> > configuring and starting MiniZooKeeperCluster more like the
> non-standalone
> > codepaths through HQuorumPeer, so that more of the codepaths are shared.
>
> I would be up for reviewing and committing a patch that did a bigger
> change whose objective was reducing the number of code paths.
> Thanks David,
> St.Ack
>

Mime
  • Unnamed multipart/alternative (inline, None, 0 bytes)
View raw message