Wail Alkowaileet has posted comments on this change.

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


Patch Set 3:

Sorry for my ultimate stupid mistakes :-)

I added to types of error handling:
1- for invalid handle format.
2- generic exception. I directed the user to check of the logs.

Should other "async" APIs report errors similarly ? Currently we just do:
out.println(e.getMessage());
e.printStackTrace(out);

P.S. I filed a JIRA for a bug. Requesting unexisted jobId will halt the whole 
query api from responding.

-- 
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: 3
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Wail Alkowaileet <wael....@gmail.com>
Gerrit-Reviewer: Jenkins <jenk...@fulliautomatix.ics.uci.edu>
Gerrit-Reviewer: Michael Blow <mb...@apache.org>
Gerrit-Reviewer: Till Westmann <ti...@apache.org>
Gerrit-Reviewer: Wail Alkowaileet <wael....@gmail.com>
Gerrit-HasComments: No

Reply via email to