Thanks for the update Keyong. Good to know (smile) On Fri, Mar 3, 2023 at 6:23 PM Keyong Zhou <[email protected]> wrote:
> 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 > > > -- Best Regards, Yu
