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 <[email protected]>
Gerrit-Reviewer: Ildar Absalyamov <[email protected]>
Gerrit-Reviewer: Jenkins <[email protected]>
Gerrit-Reviewer: Till Westmann <[email protected]>
Gerrit-HasComments: Yes

Reply via email to