[ 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