Till Westmann has posted comments on this change.

Change subject: Add result location HTTP API.
......................................................................


Patch Set 1:

For the test: I think that Michael Blow recently added some regular expression 
support to the test framework when he was writing tests for the cluster admin 
APIs. I think that that might help.

For exceptions: I think that the practice to ignore exceptions that we don't 
know how to handle is quite dangerous. I think that in many of the cases where 
that is done, we don't understand the impact on the system behavior (and if we 
do, we should document it). So I think that we need to fix that (either 
immediately or later by filing an issue) one handler at a time ...

-- 
To view, visit https://asterix-gerrit.ics.uci.edu/1178
To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: If5d691635c6003947b4eb06291f620127de1eef8
Gerrit-PatchSet: 1
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Wail Alkowaileet <wael....@gmail.com>
Gerrit-Reviewer: Jenkins <jenk...@fulliautomatix.ics.uci.edu>
Gerrit-Reviewer: Till Westmann <ti...@apache.org>
Gerrit-Reviewer: Wail Alkowaileet <wael....@gmail.com>
Gerrit-HasComments: No

Reply via email to