[GitHub] [incubator-pinot] jackjlli commented on a change in pull request #6623: LICENSE and NOTICE changes for 0.7 release

[GitBox <gi...@apache.org>]

jackjlli commented on a change in pull request #6623:
URL: https://github.com/apache/incubator-pinot/pull/6623#discussion_r585079535



##########
File path: NOTICE-binary
##########
@@ -1,14 +1,13 @@
-Apache Pinot (incubating)
-Copyright 2018 and onwards The Apache Software Foundation
-
 This product includes software developed at
 The Apache Software Foundation (http://www.apache.org/).
 
 // ------------------------------------------------------------------
 // NOTICE file corresponding to the section 4d of The Apache License,
-// Version 2.0, in this case for
+// Version 2.0, in this case for 
 // ------------------------------------------------------------------
 
+Apache Pinot (incubating)

Review comment:
       Yes, I know that. But that doesn't mean I have to follow what the script is generated. Check out other apache projects like spark:
   https://github.com/apache/spark/blob/master/NOTICE-binary
   




----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@pinot.apache.org
For additional commands, e-mail: commits-help@pinot.apache.org