ANN: hbase 0.90.0 Release Candidate 0 available for download

2010-11-15 Thread Stack
The first hbase 0.90.0 release candidate is available for download: http://people.apache.org/~stack/hbase-0.90.0-candidate-0/ HBase 0.90.0 is the major HBase release that follows 0.20.0 and the fruit of the 0.89.x development release series we've been running of late. More than 920 issues have

Re: ANN: hbase 0.90.0 Release Candidate 0 available for download

2010-11-15 Thread Ted Yu
Just a few findings when I tried to compile our 0.20.6 based code with this new release: HConstants is final class now instead of interface RowFilterInterface is gone org.apache.hadoop.hbase.io.Cell is gone org.apache.hadoop.hbase.io.RowResult is gone constructor

Re: ANN: hbase 0.90.0 Release Candidate 0 available for download

2010-11-15 Thread Todd Lipcon
Were all of the removed classes marked @Deprecated in 0.20? If not, we need to make sure they survive somehow in 0.90 IMO. On Mon, Nov 15, 2010 at 12:33 PM, Ted Yu yuzhih...@gmail.com wrote: Just a few findings when I tried to compile our 0.20.6 based code with this new release: HConstants

Re: ANN: hbase 0.90.0 Release Candidate 0 available for download

2010-11-15 Thread Ryan Rawson
That is correct, those classes were deprecated in 0.20, and now gone in 0.90. Now you will want to use HTable and Result. Also Filter.getNextKeyHint() is an implementation detail, have a look at the other filters to get a sense of what it does. On Mon, Nov 15, 2010 at 12:33 PM, Ted Yu

Re: ANN: hbase 0.90.0 Release Candidate 0 available for download

2010-11-15 Thread Jean-Daniel Cryans
I think it's worth going at greater lengths to document that kind of breakage, maybe a page in the book? Stuff like What's new and Incompatibilities that we can easily link to. J-D On Mon, Nov 15, 2010 at 12:38 PM, Ryan Rawson ryano...@gmail.com wrote: That is correct, those classes were

Re: retrieving timestamp from Result

2010-11-15 Thread Ted Yu
We used the following code to see if the Cell (pardon me for using the old term) returned from scanner was of certain timestamp: if (result.getCellValue().getTimestamp() == ts){ In 0.90, I don't see method which returns the timestamp for Result. I am wondering if the above check can

Re: ANN: hbase 0.90.0 Release Candidate 0 available for download

2010-11-15 Thread Stack
I made an issue to cover J-D's suggestion, HBASE-3236, that lists what Ted is looking for. Its pegged against 0.90.1 but if a new RC, we can include it then. St.Ack On Mon, Nov 15, 2010 at 9:23 PM, Jean-Daniel Cryans jdcry...@apache.org wrote: I think it's worth going at greater lengths to

Re: retrieving timestamp from Result

2010-11-15 Thread Ted Yu
Since there may be more than one qualifier for a column family, is it reasonable to add Result.getColumnLatest(byte[]) which returns the latest column for given family ? Thanks On Mon, Nov 15, 2010 at 2:12 PM, Ryan Rawson ryano...@gmail.com wrote: The javadoc details what to expect. You can

Re: retrieving timestamp from Result

2010-11-15 Thread Ryan Rawson
Within a Result object you are only going to get columns for a single row. So the 2 byte[] are family and qualifier. Check the javadoc. I wrote quite a bit of documentation on these APIs. Thanks, -ryan On Mon, Nov 15, 2010 at 3:42 PM, Ted Yu yuzhih...@gmail.com wrote: Since there may be more

Re: Review Request: HBase-2001: Coprocessors: Colocate user code with regions

2010-11-15 Thread stack
On 2010-10-05 23:10:58, stack wrote: src/main/java/org/apache/hadoop/hbase/client/Action.java, line 30 http://review.cloudera.org/r/876/diff/7/?file=14158#file14158line30 I took a look at the package-info.html. Very nice doc. One thought though was that the batch methods do not

Re: HRegionInfo.createRegionName()

2010-11-15 Thread Ted Yu
Can someone tell me when false should be specified for newFormat ? Thanks On Mon, Nov 15, 2010 at 2:15 PM, Stack st...@duboce.net wrote: I made an issue to cover J-D's suggestion, HBASE-3236, that lists what Ted is looking for. Its pegged against 0.90.1 but if a new RC, we can include it

Re: HRegionInfo.createRegionName()

2010-11-15 Thread Ryan Rawson
Why are you calling this method? Despite it being 'public', users are not expected to talk to the filesystem directly. Your Warranty would be void if you did so :-) -ryan On Mon, Nov 15, 2010 at 4:44 PM, Ted Yu yuzhih...@gmail.com wrote: Can someone tell me when false should be specified for

Re: Review Request: [HBASE-2321] [HBASE-2002] Add support for per-region dynamically registered RPC endpoints for coprocessors and allow configurable RPC client/server implementations

2010-11-15 Thread stack
--- This is an automatically generated e-mail. To reply, visit: http://review.cloudera.org/r/816/#review1933 --- Ship it! I did a quick pass over this. Most I'd seen already over in

Re: HRegionInfo.createRegionName()

2010-11-15 Thread Ted Yu
Ryan: It originated from the following call against 0.20.6: HRegionInfo.createRegionName(tableName, null, ZEROES); On Mon, Nov 15, 2010 at 4:48 PM, Ryan Rawson ryano...@gmail.com wrote: Why are you calling this method? Despite it being 'public', users are not expected to talk to the

Re: HRegionInfo.createRegionName()

2010-11-15 Thread Ryan Rawson
Hi, Sorry I dont have enough context here. The region name format is designed to be semi-backwards compatible, so you should be able to use 'true' for the new field and use the resulting string in searches in META even with old region names in META. -ryan On Mon, Nov 15, 2010 at 5:03 PM, Ted