Hi Yu, Thanks for the reminder, we have already fixed it :)
https://github.com/apache/incubator-celeborn/commit/9aabb43699225d47c1470027b98a42210df914e8 https://github.com/apache/incubator-celeborn/commit/dcf1e018f6352a64250c64d64e21e3eae1f8fa14 Thanks, Keyong Zhou Yu Li <[email protected]> 于2023年3月3日周五 18:03写道： > Hi all, > > Since Spark has already taken action to fix the issue, do we have a PR to > fix ours? Thanks. > > Some background for easier reference: > https://lists.apache.org/thread/6mcwfb2gmxpj1r281pd13gs35wjqfxom > > Best Regards, > Yu > > > ---------- Forwarded message --------- > From: Dongjoon Hyun <[email protected]> > Date: Thu, 2 Mar 2023 at 16:14 > Subject: Re: [Question] LimitedInputStream license issue in Spark source. > To: <[email protected]> > Cc: incubator general apache <[email protected]>, Cheng Pan < > [email protected]>, Ethan Feng <[email protected]>, Willem Jiang < > [email protected]>, <[email protected]> > > > Thank you. Here is the PR to fix that. > > https://github.com/apache/spark/pull/40249 > [SPARK-42649][CORE] Remove the standard Apache License header from the top > of third-party source files > > Dongjoon. > > > On Wed, Mar 1, 2023 at 11:53 PM <[email protected]> wrote: > > > Hi, > > > > See https://www.apache.org/legal/src-headers.html#3party - "Do not add > > the standard Apache License header to the top of third-party source > files.” > > and "Minor modifications/additions to third-party source files should > > typically be licensed under the same terms as the rest of the third-party > > source for convenience.” > > > > Kind Regards, > > Justin >
