On Tuesday, November 26, 2013 4:10:08 PM UTC+1, Jan Marten wrote: > > The inconsistency with the current RequestFactory is that if in a batch > request, a single sub-request has a constraint violation >
There's no inconsistency, because that's not how things work. 1. All objects (entities and "value objects") are "deserialized" from the request (for entities, it involves first retrieving them from the data store) 2. Then they're all validated. If there's a constraint violation, things stop here and you'll have onConstraintViolations on the client-side, in each and every Receiver attached to the RequestContext (i.e. or one of its Requests/InstanceRequests) 3. Otherwise, "invocations" are processed, in the same order they were added to the RequestContext on client side. Each invocation either succeeds or fails, and the onSuccess or onFailure will be called accordingly on the client side for the corresponding Receiver. The exception raised might be a ConstraintViolationException, it doesn't change anything: onFailure (not onConstraintViolation) will be called for the appropriate Receiver (not all receivers) 4. Then all entities (including those returned by "invocations") are checked for "liveness", to tell the client which kind of EntityProxyChange event to fire (PERSIST/UPDATE/DELETE) 5. And finally the response is constructed, with serialized objects, etc. > then onConstraintViolation is called for every sub-request and the whole > batch request fails (onFailure is called). > Whereas if in a sub-request an exception is raised on the server only for > this single sub-request onFailure is called and the surrounding > batch-request succeeds with onSuccess. > > Thus, the RequestContext cannot be reused since "reuse" is only called for > constraint violations and failures. > > Hence, as described in the original post, after a server failure the proxy > cannot be reused and the user-entered data is gone (in contrast to a > constraint violation). > Yes. Exceptions are meant to be "exceptional", you should use a "return value" to convey errors. In other words, onFailure should never be called, unless something *unpredictable* happens. -- You received this message because you are subscribed to the Google Groups "Google Web Toolkit" group. To unsubscribe from this group and stop receiving emails from it, send an email to [email protected]. To post to this group, send email to [email protected]. Visit this group at http://groups.google.com/group/google-web-toolkit. For more options, visit https://groups.google.com/groups/opt_out.
