[
https://issues.apache.org/jira/browse/HBASE-5542?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13226754#comment-13226754
]
Lars Hofhansl commented on HBASE-5542:
--------------------------------------
@Scott: Sorry missed your earlier comment. isAtomic would work.
I wonder if we'd get more flexibility if we leave this logic in the row
processor.
In the case of doMiniBatchPut, the row processor could maintain batchOp as
internal state and use that in all of pre/post/process.
Re: your last comment. Thanks for bearing with me. As I said, this is important
(and a bit tricky) to get right. Thank you very much for working on it!
I hope we won't find any holes in the latest approach.
Actually one more thing I was wondering: For the core operations, it makes
little sense to timebound this with an extra thread. It would be nice if there
was some option to enforce a timebound or leave it up to the implementer to
guarantee it won't take too long.
In fact I would probably leave the time bounding completely to the custom code.
> Unify HRegion.mutateRowsWithLocks() and HRegion.processRow()
> ------------------------------------------------------------
>
> Key: HBASE-5542
> URL: https://issues.apache.org/jira/browse/HBASE-5542
> Project: HBase
> Issue Type: Improvement
> Reporter: Scott Chen
> Assignee: Scott Chen
> Fix For: 0.96.0
>
> Attachments: HBASE-5542.D2217.1.patch, HBASE-5542.D2217.2.patch,
> HBASE-5542.D2217.3.patch
>
>
> mutateRowsWithLocks() does atomic mutations on multiple rows.
> processRow() does atomic read-modify-writes on a single row.
> It will be useful to generalize both and have a
> processRowsWithLocks() that does atomic read-modify-writes on multiple rows.
> This also helps reduce some redundancy in the codes.
--
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
