asterixdb-notifications mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Till Westmann (Code Review)" <do-not-re...@asterixdb.incubator.apache.org>
Subject Change in asterixdb[master]: Add result location HTTP API.
Date Mon, 19 Sep 2016 15:33:48 GMT
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.y.k@gmail.com>
Gerrit-Reviewer: Jenkins <jenkins@fulliautomatix.ics.uci.edu>
Gerrit-Reviewer: Till Westmann <tillw@apache.org>
Gerrit-Reviewer: Wail Alkowaileet <wael.y.k@gmail.com>
Gerrit-HasComments: No

Mime
View raw message