Build failed in Hudson: HBase-TRUNK #1302

2010-06-06 Thread Apache Hudson Server
See Changes: [apurtell] HBASE-2397 Bytes.toStringBinary escapes printable chars -- [...truncated 853 lines...] Running org.apache.hadoop.hbase.regionserver.wal.TestLogRolling Tests run: 2

Review Request: HBASE-2618 Don't inherit from HConstants.

2010-06-06 Thread Jonathan Gray
--- This is an automatically generated e-mail. To reply, visit: http://review.hbase.org/r/132/#review135 --- This patch changes a bunch of tabbing patterns to be different from what is

RE: Working towards a dev release

2010-06-06 Thread Jonathan Gray
I'll be putting up patches to jira/reviewboard when they're ready for consumption. There's no unified thing that would actually compile and make sense in a github or I would just push it out. All of my compact/split stuff was based on branch and was before the master changes which impact it so

Review Request: HBASE-2400: new connector for Avro RPC access to HBase cluster

2010-06-06 Thread Jonathan Gray
--- This is an automatically generated e-mail. To reply, visit: http://review.hbase.org/r/128/#review137 --- This looks great Hammer. No major comments. trunk/src/main/java/org/apach

Re: Review Request: HBASE-2400: new connector for Avro RPC access to HBase cluster

2010-06-06 Thread Andrew Purtell
--- This is an automatically generated e-mail. To reply, visit: http://review.hbase.org/r/128/#review138 --- Looks good Jeff. Some missing functionality IMHO. See my individual comment

Re: Review Request: HBASE-2618 Don't inherit from HConstants.

2010-06-06 Thread Benoit Sigoure
> On 2010-06-06 09:08:19, Jonathan Gray wrote: > > This patch changes a bunch of tabbing patterns to be different from what is > > currently done in the codebase. Primarily the full indenting of arguments > > to align with each other. > > > > As far as I know, this format is not used anywhere

Review Request: HBASE-2618 Don't inherit from HConstants.

2010-06-06 Thread Jonathan Gray
> On 2010-06-06 09:08:19, Jonathan Gray wrote: > > This patch changes a bunch of tabbing patterns to be different from what is > > currently done in the codebase. Primarily the full indenting of arguments > > to align with each other. > > > > As far as I know, this format is not used anywhere e

Review Request: HBASE-2618 Don't inherit from HConstants.

2010-06-06 Thread Jonathan Gray
> On 2010-06-06 09:08:19, Jonathan Gray wrote: > > This patch changes a bunch of tabbing patterns to be different from what is > > currently done in the codebase. Primarily the full indenting of arguments > > to align with each other. > > > > As far as I know, this format is not used anywhere e

HBASE-1845 - which version to target

2010-06-06 Thread Marc Limotte
I was going to upgrade the patch on HBASE-1845 to work with the latest hbase-0.20 branch and submit it through Review Board. But it looks like the changes for hbase-2066 have been removed and the comments on that issue say that it is going to be a fix for 0.21 now. Some of the hbase-1845 patch is

Re: HBASE-1845 - which version to target

2010-06-06 Thread Todd Lipcon
Hey Marc, You should target the trunk branch for this patch. HBASE-2066 was committed to trunk a while back. -Todd On Sun, Jun 6, 2010 at 3:51 PM, Marc Limotte wrote: > I was going to upgrade the patch on HBASE-1845 to work with the latest > hbase-0.20 branch and submit it through Review Board

Build failed in Hudson: HBase-TRUNK #1303

2010-06-06 Thread Apache Hudson Server
See Changes: [todd] HBASE-2676. TestInfoServers should use ephemeral ports -- [...truncated 853 lines...] Running org.apache.hadoop.hbase.regionserver.wal.TestLogRolling Tests run: 2, Fai

Review Request: HBASE-1845 MultiGet, MultiDelete, and MultiPut - batched to the appropriate region servers

2010-06-06 Thread Marc Limotte
--- This is an automatically generated e-mail. To reply, visit: http://review.hbase.org/r/151/ --- Review request for hbase. Summary --- Updated the patch to work on the trunk.