[
https://issues.apache.org/jira/browse/HBASE-5515?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Phabricator updated HBASE-5515:
-------------------------------
Attachment: HBASE-5515.D2067.17.patch
sc updated the revision "HBASE-5515 [jira] Add a processRow API that supports
atomic multiple reads and writes on a row".
Reviewers: tedyu, dhruba, JIRA
@Lars:
"They can be loaded dynamically from a jar on HDFS and per table, or
statically and globally for all tables."
This is really cool!
I made a sight change to HRegion.processRow() so that it takes a RowProcessor
interface (instead of the BaseRowProcessor class)
Now people can still define their own CorocessorProtocol that implements
RowProcessor and have the ability to dynamically load this class as you
described.
I wrote a unit test TestProcessRowWithCustomMadeProtocol.java to verify that
use case.
But if this is cool with you, I sill like to keep the ProcessRowProtocol and
ProcessRowEndpoint because that's a convenient entry point for most users.
REVISION DETAIL
https://reviews.facebook.net/D2067
AFFECTED FILES
src/main/java/org/apache/hadoop/hbase/coprocessor/ProcessRowEndpoint.java
src/main/java/org/apache/hadoop/hbase/coprocessor/ProcessRowProtocol.java
src/main/java/org/apache/hadoop/hbase/coprocessor/RowProcessor.java
src/main/java/org/apache/hadoop/hbase/regionserver/HRegion.java
src/test/java/org/apache/hadoop/hbase/coprocessor/TestProcessRowEndpoint.java
src/test/java/org/apache/hadoop/hbase/coprocessor/TestProcessRowWithCustomMadeProtocol.java
> 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.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
