Hi Justin, Thank you very much for your detailed and constructive feedback — we admit we previously misunderstood certain requirements, and your comments have greatly helped us deepen our understanding of ASF licensing policies.
We have carefully reviewed and addressed the issues you raised regarding the LICENSE and NOTICE files: 1. NOTICE file: - We have revised it to include only the original NOTICE content from upstream Apache projects (Flink, Spark, Hive, Netty, Presto, etc.), as these projects each maintain their own NOTICE files. - Guava, which does not have a NOTICE file, has been removed from our NOTICE (though it remains properly declared in the LICENSE file). 2. LICENSE file: We have restructured it to clearly specify: - The origin (project name, URL, and version) of each third-party component; - The original copyright notice from each upstream project; - The exact list of derived files in our codebase. This ensures that every third-party file is traceable to its source and properly attributed. 3. Source file headers: - All derived files (e.g., those from Netty, Flink, Spark, etc.) retain their original copyright notices and include the ASF license header for modifications, as required. We believe these changes now suitable with the ASF Licensing Policy and the guidance referenced in [1]. A new release candidate (RC3) incorporating all the above fixes is being prepared and will be posted shortly for re-vote. Thank you again for your patient and constructive review! Kind Regards, loogn On behalf of the Apache GeaFlow (incubating) community Justin Mclean <[email protected]> 于2025年11月4日周二 05:46写道： > Hi, > > > LICENSE and NOTICE files have been updated: > > > > The Netty-derived or other files (CleanerJava6.java, CleanerJava9.java, > AbstractFileRegion.java, etc.) are now properly declared in LICENSE, and > the Netty or other project's copyright notice has been added to NOTICE. > > The GeaflowMemoryViewWriterBenchmark.java file now correctly contains > ASF header. > > The mock file flameGraphContent.ts was confirmed as project-original > content and now includes the standard ASF header. > > The intellij-java-style.xml file is confirmed as project-authored and > now includes an XML-formatted ASF license header. > I think you may not understand what needs to go into the NOTICE file; it’s > not a place to list all copyrights. Only certain things need to be placed > there. If this case any bundled software under the Apache license and has a > NOTICE file, then you must include that NOTICE file or relevant parts of it > in your NOTICE file. If it doesn’t have a NOTICE file, then there's no need > to include anything. If the copyright statement has been removed (with the > owner's permission) from files, then that also goes into the NOTICE file. > See [1] > > Kind Regards, > Justin > > 1. https://infra.apache.org/licensing-howto.html
