asterixdb-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Ian Maxon (Code Review)" <do-not-re...@asterix-gerrit.ics.uci.edu>
Subject Change in hyracks[master]: Fix source packaging, address misc. release issues
Date Wed, 09 Sep 2015 04:59:31 GMT
Ian Maxon has posted comments on this change.

Change subject: Fix source packaging, address misc. release issues
......................................................................


Patch Set 1:

(2 comments)

https://asterix-gerrit.ics.uci.edu/#/c/381/1/LICENSE
File LICENSE:

Line 10: ---
> I'm sorry I was unclear on this int he e-mail. But I think that the license
Ooooh ok, sorry, the svn example is very clear. I'll fix that.


https://asterix-gerrit.ics.uci.edu/#/c/381/1/pom.xml
File pom.xml:

Line 156:            <exclude>**/actual/customer_result/part-*</exclude>
> Aren't those files just around when the tests have been run? Are they reall
They (along with others in here) are only around when the tests get run, but if RAT is part
of verify, it's nearly certain they'll be around.


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

Gerrit-MessageType: comment
Gerrit-Change-Id: I98b18f24a20dcd8dc75e828e47fb0ab88179a4ef
Gerrit-PatchSet: 1
Gerrit-Project: hyracks
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: Till Westmann <tillw@apache.org>
Gerrit-HasComments: Yes

Mime
View raw message