[ 
https://issues.apache.org/jira/browse/HBASE-2400?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12876931#action_12876931
 ] 

HBase Review Board commented on HBASE-2400:
-------------------------------------------

Message from: "Jeff Hammerbacher" <[email protected]>


bq.  On 2010-06-06 10:23:33, Andrew Purtell wrote:
bq.  > trunk/src/main/java/org/apache/hadoop/hbase/avro/hbase.genavro, line 181
bq.  > <http://review.hbase.org/r/128/diff/1/?file=979#file979line181>
bq.  >
bq.  >     No support for filters. For REST, the model classes recursively walk 
the filter structure and build a JSON representation that is then passed as a 
string.
bq.  >     
bq.  >     No support for setBatch()
bq.  >     
bq.  >     Consider support for setCacheBlocks() (regionserver level caching) 
and setCaching() (connector level caching)?
bq.  
bq.  Jeff Hammerbacher wrote:
bq.      https://issues.apache.org/jira/browse/HBASE-2690 is for the caching 
and https://issues.apache.org/jira/browse/HBASE-2687 is for the filters. Thanks 
for letting me know how you did things in the REST interface; I'll try to mimic 
your approach. I held off on implementing because I wasn't quite sure what the 
API would look like.

Doesn't make sense with my current Scan API (super simple: table + number of 
rows). I filed https://issues.apache.org/jira/browse/HBASE-2705 to take care of 
this one at a later date.


bq.  On 2010-06-06 10:23:33, Andrew Purtell wrote:
bq.  > trunk/src/main/java/org/apache/hadoop/hbase/avro/AvroServer.java, line 
386
bq.  > <http://review.hbase.org/r/128/diff/1/?file=950#file950line386>
bq.  >
bq.  >     Export exists() to the client.

Done


bq.  On 2010-06-06 10:23:33, Andrew Purtell wrote:
bq.  > trunk/src/main/java/org/apache/hadoop/hbase/avro/AvroServer.java, line 
335
bq.  > <http://review.hbase.org/r/128/diff/1/?file=950#file950line335>
bq.  >
bq.  >     No split()?

Done


bq.  On 2010-06-06 10:23:33, Andrew Purtell wrote:
bq.  > trunk/src/main/java/org/apache/hadoop/hbase/avro/AvroServer.java, line 
324
bq.  > <http://review.hbase.org/r/128/diff/1/?file=950#file950line324>
bq.  >
bq.  >     That would be good.

Removed comment; this will work with region names as well.


- Jeff


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://review.hbase.org/r/128/#review138
-----------------------------------------------------------





> new connector for Avro RPC access to HBase cluster
> --------------------------------------------------
>
>                 Key: HBASE-2400
>                 URL: https://issues.apache.org/jira/browse/HBASE-2400
>             Project: HBase
>          Issue Type: Task
>          Components: avro
>            Reporter: Andrew Purtell
>            Priority: Minor
>         Attachments: HBASE-2400-v0.patch
>
>
> Build a new connector contrib architecturally equivalent to the Thrift 
> connector, but using Avro serialization and associated transport and RPC 
> server work. Support AAA (audit, authentication, authorization). 

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

Reply via email to