[
https://issues.apache.org/jira/browse/HBASE-5515?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13225287#comment-13225287
]
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".
@jyates: Thanks for the review comments. I think what you and Lars think
makes sense. I guess I started this patch by using RowProcessor a parameter of
HTable. So I feel RowProcessor should be a parameter. But with the Coprocessor
case, it seems what you guys think makes more sense.
I will update this patch soon.
INLINE COMMENTS
src/main/java/org/apache/hadoop/hbase/coprocessor/ProcessRowEndpoint.java:52
The abstract method approach is a good idea. Thanks!
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.10.patch,
> HBASE-5515.D2067.11.patch, HBASE-5515.D2067.12.patch,
> HBASE-5515.D2067.13.patch, HBASE-5515.D2067.14.patch,
> HBASE-5515.D2067.15.patch, HBASE-5515.D2067.16.patch,
> HBASE-5515.D2067.17.patch, HBASE-5515.D2067.18.patch,
> HBASE-5515.D2067.19.patch, HBASE-5515.D2067.2.patch,
> HBASE-5515.D2067.3.patch, HBASE-5515.D2067.4.patch, HBASE-5515.D2067.5.patch,
> HBASE-5515.D2067.6.patch, HBASE-5515.D2067.7.patch, HBASE-5515.D2067.8.patch,
> HBASE-5515.D2067.9.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
