Re: Review for HBASE-7568 [replication] Create an interface for replication queues

2013-03-28 Thread Chris Trezzo
Thanks J-D! On Thu, Mar 28, 2013 at 2:10 PM, Jean-Daniel Cryans wrote: > Two general comments about the patch. The code itself looks good. > > J-D > > On Thu, Mar 28, 2013 at 1:40 PM, Chris Trezzo wrote: > > Hi All, > > > > I recently posted a patch for HBASE-7568 and it would be great if I coul

Re: Review for HBASE-7568 [replication] Create an interface for replication queues

2013-03-28 Thread Jean-Daniel Cryans
Two general comments about the patch. The code itself looks good. J-D On Thu, Mar 28, 2013 at 1:40 PM, Chris Trezzo wrote: > Hi All, > > I recently posted a patch for HBASE-7568 and it would be great if I could > get some feedback/eyes on it. If you need a higher-level context, the > parent issu

Review for HBASE-7568 [replication] Create an interface for replication queues

2013-03-28 Thread Chris Trezzo
Hi All, I recently posted a patch for HBASE-7568 and it would be great if I could get some feedback/eyes on it. If you need a higher-level context, the parent issue HBASE-7564 has a brief document that outlines the intended class structure of the refactor. I have split the refactor into multiple p