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]: Run ExecutionTest w/integration test framework
Date Thu, 03 Dec 2015 02:36:38 GMT
Ian Maxon has posted comments on this change.

Change subject: Run ExecutionTest w/integration test framework
......................................................................


Patch Set 10:

99.9% of this by volume is just changes to rename nc1/2 to asterix_nc1/2 . The reason for
that is there's no way to actually name an NC in managix as far as I can tell, that won't
end up including the instance name. 

The real meat is in AsterixInstallerIntegrationUtil, ClusterExecutionIT,ManagixExecutionIT,
and so on. There's only one questionable test case change (imo) and that's mentioned in the
comment. It still doesn't seem to pass though so this may be revealing some underlying issue.

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

Gerrit-MessageType: comment
Gerrit-Change-Id: I423f2a7c77839b999d466dd5cace302574d956c0
Gerrit-PatchSet: 10
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Ian Maxon <imaxon@apache.org>
Gerrit-Reviewer: Chris Hillery <ceej@lambda.nu>
Gerrit-Reviewer: Ian Maxon <imaxon@apache.org>
Gerrit-Reviewer: Jenkins <jenkins@fulliautomatix.ics.uci.edu>
Gerrit-Reviewer: Till Westmann <tillw@apache.org>
Gerrit-Reviewer: Yingyi Bu <buyingyi@gmail.com>
Gerrit-HasComments: No

Mime
View raw message