-Re: removing lock ids from RegionObserver
Anoop John 2013-07-07, 05:04
+1 from me for changing the CP signature by removing lockids.. CP will be
able to acquire lock on rows? (lockRow will be still public?)
On Sun, Jul 7, 2013 at 3:34 AM, Ted Yu <[EMAIL PROTECTED]> wrote:
> +1 on the removal of lock Ids.
> On Sat, Jul 6, 2013 at 2:49 PM, Dave Latham <[EMAIL PROTECTED]> wrote:
> > In HBASE-8877 I'm proposing to change row locks to be reentrant and
> > based rather than tracking lock ids. Part of that change would mean
> > altering the RegionObserver coprocessor. Specifically, the methods
> > preBatchMutate and postBatchMutate would have the parameter
> > MiniBatchOperationInProgress<Pair<Mutation,
> > Integer>> miniBatchOp become
> > MiniBatchOperationInProgress<Mutation> miniBatchOp
> > Does anyone rely on lock ids for this coprocessor or object to the
> > Is it safe to make this incompatible change for 0.96?
> > Thanks, Andrew, for already chiming in on the issue.
> > For more background, see that issue and discussion on HBASE-8806.
> > Dave