asterixdb-notifications mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Ildar Absalyamov (Code Review)" <do-not-re...@asterixdb.incubator.apache.org>
Subject Change in asterixdb[master]: Re-enable and fix test result for ASTERIXDB-1211
Date Sat, 19 Mar 2016 01:11:48 GMT
Ildar Absalyamov has posted comments on this change.

Change subject: Re-enable and fix test result for ASTERIXDB-1211
......................................................................


Patch Set 3:

(1 comment)

https://asterix-gerrit.ics.uci.edu/#/c/734/3/asterix-app/src/test/resources/metadata/testsuite.xml
File asterix-app/src/test/resources/metadata/testsuite.xml:

Line 220:     </test-case>
> It seems that we have no directories meta22 and meta23 in the source tree. 
Yeah it happened due to the conflict with https://asterix-gerrit.ics.uci.edu/#/c/558/15/asterix-app/src/test/resources/metadata/testsuite.xml
But it's all good the test case just got renamed. There is no regression or drop in coverage
(you can check the jacoco report since it's been integrated into the build ;))


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

Gerrit-MessageType: comment
Gerrit-Change-Id: I67c6cc748bf49b3de44b2ca84a3a53f9c941170c
Gerrit-PatchSet: 3
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Ian Maxon <imaxon@apache.org>
Gerrit-Reviewer: Ildar Absalyamov <ildar.absalyamov@gmail.com>
Gerrit-Reviewer: Jenkins <jenkins@fulliautomatix.ics.uci.edu>
Gerrit-Reviewer: Till Westmann <tillw@apache.org>
Gerrit-HasComments: Yes

Mime
View raw message