Re: [apps-discuss] JSON patch: "test" operation

Hi, All

I think test would make it complicated. The test could result in <, >, <=, >=, = conditions.

If we want something like HTTP If-Match mechanism, I would think it should not be in the JSON patch. We should keep JSON patch simple. The status code is better to be delivered in the protocol layer who delivers the JSON patch document.

Cheers,
Linyi

From: apps-discuss-bounces@ietf.org [mailto:apps-discuss-bounces@ietf.org] On Behalf Of mike amundsen
Sent: Thursday, December 01, 2011 1:27 AM
To: Paul C. Bryan
Cc: IETF Apps Discuss
Subject: Re: [apps-discuss] JSON patch: "test" operation

Julian:

Is the thinking that _clients_ should tell servers to test an insert/update?  IOW, clients would want to see test results?  A "dry run" scenario?

Also, was there any talk about how servers would handle failed tests (HTTP Response code, etc.)? If one or more tests fail, would the server have the option of making the changes and then returning a list of success/fail information?

mca
http://amundsen.com/blog/
http://twitter.com@mamund
http://mamund.com/foaf.rdf#me

On Wed, Nov 30, 2011 at 12:09, Paul C. Bryan <paul.bryan@forgerock.com<mailto:paul.bryan@forgerock.com>> wrote:
Yes, assert/equals/test has been mentioned in the past. As you point out, HTTP preconditions to handle this to some extent. I've avoided adding it thus far mostly because I haven't had a single concrete use case outside of HTTP. Given at least the pattern of multiple requests wanting to test a value for equality, I think I have enough data to add it to the next revision of the spec. Consider it added.

Paul

On Wed, 2011-11-30 at 16:22 +0100, Julian Reschke wrote:

Hi,

this came up during discussions of patch formats on the Apache

jackrabbit-dev mailing list:

Would it make sense to add a "test" operation that allows checking the

state of the JSON object to be modified?

It would be consistent with other diff formats that use context

information in order to check whether the patch "cleanly applies".

An example would be:

    [

          { "test": "/vsn", "value" : 17 },

          { "replace": "/vsn", "value" : 18 },

          { "replace": "/balance", "value": 1234 }

    ]

And yes, concurrency control can also be achieved using conditional HTTP

methods, but that doesn't mean that they don't make sense in the patch

format as well.

Best regards, Julian

_______________________________________________

apps-discuss mailing list

apps-discuss@ietf.org<mailto:apps-discuss@ietf.org>

https://www.ietf.org/mailman/listinfo/apps-discuss

_______________________________________________
apps-discuss mailing list
apps-discuss@ietf.org<mailto:apps-discuss@ietf.org>
https://www.ietf.org/mailman/listinfo/apps-discuss