Michael Blow has posted comments on this change.

Change subject: AsterixDB Rat Execution Audit & Fixes
......................................................................


Patch Set 8:

(3 comments)

https://asterix-gerrit.ics.uci.edu/#/c/1609/8/asterixdb/asterix-app/src/test/resources/runtimets/only_it.xml
File asterixdb/asterix-app/src/test/resources/runtimets/only_it.xml:

PS8, Line 1: xml
> add license header?
this should be an empty file- accidentally submitted.  I can add a header to 
the empty file, but couldn't see how to easily output it when we generate the 
error.xml file intended to copy here.  I guess the foundation would accept not 
having a copyright on an empty file?


https://asterix-gerrit.ics.uci.edu/#/c/1609/8/asterixdb/asterix-experiments/src/main/resources/gantt/foo.eps
File asterixdb/asterix-experiments/src/main/resources/gantt/foo.eps:

PS8, Line 5:  
> Can we/should we remove this WS?
Done


https://asterix-gerrit.ics.uci.edu/#/c/1609/8/asterixdb/asterix-experiments/src/main/resources/gantt/gantt.py
File asterixdb/asterix-experiments/src/main/resources/gantt/gantt.py:

PS8, Line 5:  
> Can we/should we remove this WS?
Done


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

Gerrit-MessageType: comment
Gerrit-Change-Id: I3d771c5b63eaef4b6a0b4da769039ed5bca17736
Gerrit-PatchSet: 8
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Michael Blow <[email protected]>
Gerrit-Reviewer: Jenkins <[email protected]>
Gerrit-Reviewer: Michael Blow <[email protected]>
Gerrit-Reviewer: Till Westmann <[email protected]>
Gerrit-Reviewer: abdullah alamoudi <[email protected]>
Gerrit-HasComments: Yes

Reply via email to