vlsi commented on a change in pull request #1633: Fix copyright headers, move copyright owner from Resources.java to NOTICE URL: https://github.com/apache/calcite/pull/1633#discussion_r354015817
########## File path: core/src/main/java/org/apache/calcite/runtime/Resources.java ########## @@ -1,10 +1,10 @@ /* - * Licensed to Julian Hyde under one or more contributor license Review comment: @julianhyde , AFAIK, we must not retain the original copyright owners in the source file, rather we move them to the NOTICE file (see https://www.apache.org/legal/src-headers.html#headers ) ---------------------------------------------------------------- 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: [email protected] With regards, Apache Git Services
