Since both license headers are Apache License 2.0, we don't see any issue there. They are compatible.
The first line of the second license header means the file was copied from Google Guava project originally. Apache Spark community keeps the original header because it has `Authorship` part, `Copyright (C) 2007 The Guava Authors`. /* * Based on LimitedInputStream.java from Google Guava * * Copyright (C) 2007 The Guava Authors * * Licensed under the Apache License, Version 2.0 (the "License"); * you may not use this file except in compliance with the License. * You may obtain a copy of the License at * * http://www.apache.org/licenses/LICENSE-2.0 * * Unless required by applicable law or agreed to in writing, software * distributed under the License is distributed on an "AS IS" BASIS, * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. * See the License for the specific language governing permissions and * limitations under the License. */ Dongjoon. On Tue, Feb 28, 2023 at 10:27 PM Ethan Feng <ethanf...@apache.org> wrote: > Hi developers, > I have been running a release in project celeborn(incubating) > recently and the incubator PPMCs have arguments[2] about the headers > in source file[1] that we copied from the Spark source. > We don't understand why the source file[1] has two headers with > different copyright disclaimers. Could you explain why the source > file[1] needs two headers? > > Regards, > Ethan Feng > > --------------------------------------------------------------------- > 1. org/apache/spark/network/util/LimitedInputStream.java > 2. https://lists.apache.org/thread/q8hchs4n2twhjl0qf3rn0ln0hyrvrnnp > > --------------------------------------------------------------------- > To unsubscribe e-mail: dev-unsubscr...@spark.apache.org > >
