Hi

 

Currently the checkAndPut and checkAndDelete api internally calls the
internalPut and internalDelete.  May be we can just call doMiniBatchMutation
only.  This will help in future like if we have some hooks and the CP
handles certain cases in the doMiniBatchMutation the same can be done while
doing a put thro checkAndPut or while doing a delete thro checkAndDelete.

 

If we are fine with this I can raise a JIRA and submit a patch which should
be a simple one.

 

Regards

Ram

Please go ahead.

On Aug 30, 2012, at 2:53 AM, "Ramkrishna.S.Vasudevan" <[EMAIL PROTECTED]> wrote:

> Hi
>
>
>
> Currently the checkAndPut and checkAndDelete api internally calls the
> internalPut and internalDelete.  May be we can just call doMiniBatchMutation
> only.  This will help in future like if we have some hooks and the CP
> handles certain cases in the doMiniBatchMutation the same can be done while
> doing a put thro checkAndPut or while doing a delete thro checkAndDelete.
>
>
>
> If we are fine with this I can raise a JIRA and submit a patch which should
> be a simple one.
>
>
>
> Regards
>
> Ram
>

Ram,

+1

It would be great if the refactor made adding a "checkAndMututate" method
trivial to add.  This method would do an  atomic check and set that and if
successful allowed for atomic deletions and puts on the same row.  (current
api only allow checkAndDelete, checkAndPut, and batchMutate).

Jon.

On Thu, Aug 30, 2012 at 2:53 AM, Ramkrishna.S.Vasudevan <
[EMAIL PROTECTED]> wrote:

> Hi
>
>
>
> Currently the checkAndPut and checkAndDelete api internally calls the
> internalPut and internalDelete.  May be we can just call
> doMiniBatchMutation
> only.  This will help in future like if we have some hooks and the CP
> handles certain cases in the doMiniBatchMutation the same can be done while
> doing a put thro checkAndPut or while doing a delete thro checkAndDelete.
>
>
>
> If we are fine with this I can raise a JIRA and submit a patch which should
> be a simple one.
>
>
>
> Regards
>
> Ram
>
>
--
// Jonathan Hsieh (shay)
// Software Engineer, Cloudera
// [EMAIL PROTECTED]

+1 Jon's suggestion.

On Thu, Aug 30, 2012 at 3:52 PM, Jonathan Hsieh <[EMAIL PROTECTED]> wrote:

> Ram,
>
> +1
>
> It would be great if the refactor made adding a "checkAndMututate" method
> trivial to add.  This method would do an  atomic check and set that and if
> successful allowed for atomic deletions and puts on the same row.  (current
> api only allow checkAndDelete, checkAndPut, and batchMutate).
>
> Jon.
>
> On Thu, Aug 30, 2012 at 2:53 AM, Ramkrishna.S.Vasudevan <
> [EMAIL PROTECTED]> wrote:
>
> > Hi
> >
> >
> >
> > Currently the checkAndPut and checkAndDelete api internally calls the
> > internalPut and internalDelete.  May be we can just call
> > doMiniBatchMutation
> > only.  This will help in future like if we have some hooks and the CP
> > handles certain cases in the doMiniBatchMutation the same can be done
> while
> > doing a put thro checkAndPut or while doing a delete thro checkAndDelete.
> >
> >
> >
> > If we are fine with this I can raise a JIRA and submit a patch which
> should
> > be a simple one.
> >
> >
> >
> > Regards
> >
> > Ram
> >
> >
>
>
> --
> // Jonathan Hsieh (shay)
> // Software Engineer, Cloudera
> // [EMAIL PROTECTED]
>

--
Best regards,

   - Andy

Problems worthy of attack prove their worth by hitting back. - Piet Hein
(via Tom White)