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

Phabricator commented on HBASE-5515:
------------------------------------

sc has commented on the revision "HBASE-5515 [jira] Add a processRow API that 
supports atomic multiple reads and writes on a row".

  Lars: Thanks for the feedback.

  Yes, the RowProcessor implementation needs to exist on the RegionServer. In 
our case, it's some predefined logic that we deployed on the server side.

  We also thought about using coprocessor also but like you said it will need 
to expose lots of details of HRegion. So it's not very easy.

  I think you have really good points about the API. We should be very careful 
about adding API because we can't take them back. The API should be as simple 
as possible.

  For our use case, this one is necessary because it allows execution of 
multiple commands on the server. This is very important for the performance. 
And I personally think the abstraction level is high enough to be a public API. 
What do you think?

REVISION DETAIL
  https://reviews.facebook.net/D2067

                
> Add a processRow API that supports atomic multiple reads and writes on a row
> ----------------------------------------------------------------------------
>
>                 Key: HBASE-5515
>                 URL: https://issues.apache.org/jira/browse/HBASE-5515
>             Project: HBase
>          Issue Type: New Feature
>            Reporter: Scott Chen
>            Assignee: Scott Chen
>         Attachments: HBASE-5515.D2067.1.patch, HBASE-5515.D2067.2.patch, 
> HBASE-5515.D2067.3.patch, HBASE-5515.D2067.4.patch
>
>
> We have modified HRegion.java internally to do some atomic row processing. It 
> will be nice to have a plugable API for this.

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: 
https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

Reply via email to