0) The DemoClient.cpp (versions, see [a]) does not compile on FC8 with
thrift 0.5.

I modified DemoClient.cpp to make it work. I've attached a diff. (Compiles
on OS
X Snow Leopard and FC 8 (EC2 instance))

[a]
HBase Change Log
Release 0.89.20100924 - Fri Sep 24 13:51:36 PDT 2010

1) Question about Hbase Thrift api:

>From Hbase.thrift, the scannerGet returns a list of TRowResult but the
description says it returns the /current/ row - are these the versions of
the
row?  If so, having tested DemoClient.cpp, for each scannerGet the result is
of
length 1 (the latest version).

So why an array of TRowResult as opposed to just TRowResult ?
  /**
   * Returns the scanner's current row value and advances to the next
   * row in the table.  When there are no more rows in the table, or a key
   * greater-than-or-equal-to the scanner's specified stopRow is reached,
   * an empty list is returned.
   *
   * @return a TRowResult containing the current row and a map of the
columns to TCells.
   *
   * @throws IllegalArgument if ScannerID is invalid
   *
   * @throws NotFound when the scanner reaches the end
   */
  list<TRowResult> scannerGet(
    /** id of a scanner returned by scannerOpen */
    1:ScannerID id
  ) throws (1:IOError io, 2:IllegalArgument ia)

Cheers
Joy

On Sun, Nov 21, 2010 at 12:04 AM, Saptarshi Guha
<[EMAIL PROTECTED]> wrote:
> 0) The DemoClient.cpp (versions, see [a]) does not compile on FC8 with
> thrift 0.5.
> I modified DemoClient.cpp to make it work. I've attached a diff. (Compiles
> on OS
> X Snow Leopard and FC 8 (EC2 instance))
> [a]
> HBase Change Log
> Release 0.89.20100924 - Fri Sep 24 13:51:36 PDT 2010
>

Sorry. Your attachment did not come across (I believe this list strips
attachments).  Would you mind posting to an issue?

>
> 1) Question about Hbase Thrift api:
> From Hbase.thrift, the scannerGet returns a list of TRowResult but the
> description says it returns the /current/ row - are these the versions of
> the
> row?  If so, having tested DemoClient.cpp, for each scannerGet the result is
> of
> length 1 (the latest version).
> So why an array of TRowResult as opposed to just TRowResult ?
>

Yes.  That seems a little silly.

Our thrift has been redone.  We should be committing HBASE-3117 to
TRUNK soon.  Hopefully that makes a little more sense than old API.

St.Ack
>   /**
>    * Returns the scanner's current row value and advances to the next
>    * row in the table.  When there are no more rows in the table, or a key
>    * greater-than-or-equal-to the scanner's specified stopRow is reached,
>    * an empty list is returned.
>    *
>    * @return a TRowResult containing the current row and a map of the
> columns to TCells.
>    *
>    * @throws IllegalArgument if ScannerID is invalid
>    *
>    * @throws NotFound when the scanner reaches the end
>    */
>   list<TRowResult> scannerGet(
>     /** id of a scanner returned by scannerOpen */
>     1:ScannerID id
>   ) throws (1:IOError io, 2:IllegalArgument ia)
>
>
> Cheers
> Joy
>