hbase-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "ryan rawson (JIRA)" <j...@apache.org>
Subject [jira] Commented: (HBASE-1292) php thrift's getRow() would throw an exception if the row does not exist
Date Sun, 05 Apr 2009 07:09:12 GMT

    [ https://issues.apache.org/jira/browse/HBASE-1292?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12695814#action_12695814

ryan rawson commented on HBASE-1292:

Using exceptions instead of out of band null values and a simple if-else test is not ideal.
 I'd like to see if at some point in the future we can eschew exceptions and only use them
for real exceptional events.

> php thrift's getRow() would throw an exception if the row does not exist
> ------------------------------------------------------------------------
>                 Key: HBASE-1292
>                 URL: https://issues.apache.org/jira/browse/HBASE-1292
>             Project: Hadoop HBase
>          Issue Type: Bug
>          Components: thrift
>    Affects Versions: 0.19.0, 0.19.1
>            Reporter: Rong-En Fan
>            Assignee: Rong-En Fan
>            Priority: Minor
>         Attachments: 1292-0.19.patch, 1292.patch
> I've been played with thrift recently, and observed an unexpected behavior: when getRow()
encounters an non-existent row key, it throws an exception like this:
> PHP Fatal error:  Uncaught exception 'Exception' with message 'getRow failed: unknown
result' in pear/thrift/packages/Hbase/Hbase.php:715
> Stack trace:
> #0 pear/thrift/packages/Hbase/Hbase.php(666): HbaseClient->recv_getRow()
> #1 htdocs/hbase/DemoClient.php(174): HbaseClient->getRow('demo_table', '00100-XXXX')
> #2 {main} thrown in pear/thrift/packages/Hbase/Hbase.php on line 715
> I would expect when we pass a non-existent key, it can throw something like NotFound
(as in scanner) or one can test with RowResult.isEmpty() just like in java api.

This message is automatically generated by JIRA.
You can reply to this email to add a comment to the issue online.

View raw message