asterixdb-notifications mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Ian Maxon (Code Review)" <do-not-re...@asterixdb.incubator.apache.org>
Subject Change in asterixdb[master]: Fix ASTERIXDB-1089
Date Tue, 24 Nov 2015 21:12:03 GMT
Ian Maxon has posted comments on this change.

Change subject: Fix ASTERIXDB-1089
......................................................................


Patch Set 1:

I'm not sure if this might've gotten fixed elsewhere, but basically the error here is if there
aren't any substitute nodes in the cluster config, this method would barf an NPE instead of
not finding anything and throwing the exception it's suppose to.

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

Gerrit-MessageType: comment
Gerrit-Change-Id: I8977d91ea82cd3a0538a84dc6d727d94fe2ece1d
Gerrit-PatchSet: 1
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Ian Maxon <imaxon@apache.org>
Gerrit-Reviewer: Ian Maxon <imaxon@apache.org>
Gerrit-Reviewer: Jenkins <jenkins@fulliautomatix.ics.uci.edu>
Gerrit-Reviewer: Keren-Audrey Ouaknine <kereno@gmail.com>
Gerrit-Reviewer: Till Westmann <tillw@apache.org>
Gerrit-Reviewer: abdullah alamoudi <bamousaa@gmail.com>
Gerrit-HasComments: No

Mime
View raw message