Till Westmann has posted comments on this change.

Change subject: [NO ISSUE][FAIL] Fix Message Propagation in HyracksException
......................................................................


Patch Set 5:

(1 comment)

https://asterix-gerrit.ics.uci.edu/#/c/2720/5/hyracks-fullstack/hyracks/hyracks-api/src/main/java/org/apache/hyracks/api/util/ErrorMessageUtil.java
File 
hyracks-fullstack/hyracks/hyracks-api/src/main/java/org/apache/hyracks/api/util/ErrorMessageUtil.java:

PS5, Line 118: getCauseMessage
We have a another solution for this in 
https://github.com/apache/asterixdb/blob/4ba23de669748c6a27a6b86d4a63241365778dc3/asterixdb/asterix-app/src/main/java/org/apache/asterix/api/http/server/ResultUtil.java#L123
which takes a more specific approach.


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

Gerrit-MessageType: comment
Gerrit-Change-Id: If167755a000e60ca082bd1cc4692c747a502f6ae
Gerrit-PatchSet: 5
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Murtadha Hubail <[email protected]>
Gerrit-Reviewer: Anon. E. Moose #1000171
Gerrit-Reviewer: Jenkins <[email protected]>
Gerrit-Reviewer: Michael Blow <[email protected]>
Gerrit-Reviewer: Murtadha Hubail <[email protected]>
Gerrit-Reviewer: Till Westmann <[email protected]>
Gerrit-Reviewer: abdullah alamoudi <[email protected]>
Gerrit-HasComments: Yes

Reply via email to