Ian Maxon has posted comments on this change. Change subject: Change License Headers ......................................................................
Patch Set 4: (2 comments) https://asterix-gerrit.ics.uci.edu/#/c/388/4/LICENSE.txt File LICENSE.txt: Line 308: For portions of the Asterix API examples at: > A newline between the sections? Sure. https://asterix-gerrit.ics.uci.edu/#/c/388/4/asterix-om/src/main/java/org/apache/asterix/om/base/temporal/GregorianCalendarSystem.java File asterix-om/src/main/java/org/apache/asterix/om/base/temporal/GregorianCalendarSystem.java: Line 46 > Is this (does it need to be) in the NOTICE? Ohh, Hmm, I was thinking since it was apache licensed it wasn't an issue. But I guess the copyright is what matters. I'll add it in. -- To view, visit https://asterix-gerrit.ics.uci.edu/388 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: comment Gerrit-Change-Id: I09bc388089e515d7f51fd39c31bfbbc9f00cf84f Gerrit-PatchSet: 4 Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Owner: Ian Maxon <[email protected]> Gerrit-Reviewer: Chris Hillery <[email protected]> Gerrit-Reviewer: Ian Maxon <[email protected]> Gerrit-Reviewer: Jenkins <[email protected]> Gerrit-Reviewer: Till Westmann <[email protected]> Gerrit-HasComments: Yes
