Re: [apps-discuss] I-D Action: draft-ietf-appsawg-json-patch-01.txt

I agree with Murray's points.

Paul

On Mon, 2012-03-12 at 20:55 +0000, Murray S. Kucherawy wrote:

> > -----Original Message-----
> > From: apps-discuss-bounces@ietf.org [mailto:apps-discuss-bounces@ietf.org] On Behalf Of Vadim Zaliva
> > Sent: Saturday, March 10, 2012 10:33 PM
> > To: apps-discuss@ietf.org
> > Subject: Re: [apps-discuss] I-D Action: draft-ietf-appsawg-json-patch-01.txt
> > 
> > I would like to suggest some extensions to this draft. I was trying to
> > use proposed syntax in one of my project and found it lacking a couple
> > of operations. Please find below my proposed extensions:
> > 
> > Operation "add-unique"
> > 
> > The new value appended the array only if the it presently does not
> > contain an element with such value. If an element with such value
> > already exists, the duplicate will not be added.
> 
> This feels like feature creep to me.  A patch to me is a sequence of steps for moving some content from one known state to another.  This proposed capability allows one to move from an uncertain state to a known state.  That might be a useful thing to do, but it's not something I think of as a "patch".
> 
> > Operation "remove-all"
> > 
> > This operations removes from array all elements with given value.
> 
> I have the same concern here.
> 
> > Operation "replace-all"
> > 
> > This operations replaces in an array all elements with given value with
> > a new value.
> 
> And here.
> 
> -MSK
> _______________________________________________
> apps-discuss mailing list
> apps-discuss@ietf.org
> https://www.ietf.org/mailman/listinfo/apps-discuss