Ian Maxon has posted comments on this change.

Change subject: [ASTERIXDB-2388] Add nodeIDs to cluster state query failure 
exception
......................................................................


Patch Set 4:

(1 comment)

https://asterix-gerrit.ics.uci.edu/#/c/2654/4/asterixdb/asterix-algebra/src/main/java/org/apache/asterix/translator/AbstractLangTranslator.java
File 
asterixdb/asterix-algebra/src/main/java/org/apache/asterix/translator/AbstractLangTranslator.java:

PS4, Line 84: AsterixException
> Could we introduce an error code here and use AsterixException.create to cr
There is an error code that fits this but the issue is that we print the 
exception message based on just the error code in that case.


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

Gerrit-MessageType: comment
Gerrit-Change-Id: I8684f01c483595f1488537d82335d79f173f1517
Gerrit-PatchSet: 4
Gerrit-Project: asterixdb
Gerrit-Branch: release-0.9.4-pre-rc
Gerrit-Owner: Ian Maxon <[email protected]>
Gerrit-Reviewer: Anon. E. Moose #1000171
Gerrit-Reviewer: Ian Maxon <[email protected]>
Gerrit-Reviewer: Jenkins <[email protected]>
Gerrit-Reviewer: Michael Blow <[email protected]>
Gerrit-Reviewer: Till Westmann <[email protected]>
Gerrit-HasComments: Yes

Reply via email to