[CANCEL][VOTE] Release Apache StreamPark 2.1.4-RC1

2024-04-29 Thread Shaokang Lv
Hello Incubator Community: I'm cancelling this vote: https://lists.apache.org/thread/6z7ryxbd59tc0f7fjpblpfjy2nwnxwp4 Because of License issues. I'll fix them and start the new round vote process. Thanks a lot for all your help. Best, Shaokang Lv

Re: [VOTE] Release Apache StreamPark(Incubating) 2.1.4-rc1

2024-04-29 Thread shaokang lv
Thanks for your review, this vote is closed. Best, Shaokang Lv Huajie Wang 于2024年4月29日周一 18:17写道: > Thanks for your review work with profound respect. we have always > attached great importance to License compliance issues. We will close > the current vote later and initiate a new round of

Re: [DISCUSS] Drop the incubator- prefix for podling's GitHub repo

2024-04-29 Thread Xinyu Zhou
If the `incubator-` prefix significantly burdens the graduation process or introduces inconveniences in package or dependency management, its removal might be considered. However, IMO, maintaining incubating in the description, and website, especially in the *release version*, clearly signals

Re: [VOTE] Release Apache StreamPark(Incubating) 2.1.4-rc1

2024-04-29 Thread Huajie Wang
Thanks for your review work with profound respect. we have always attached great importance to License compliance issues. We will close the current vote later and initiate a new round of voting. Best, Huajie Wang Justin Mclean 于2024年4月29日周一 17:52写道: > Hi, > > > Thanks for your

Re: [VOTE] Release Apache StreamPark(Incubating) 2.1.4-rc1

2024-04-29 Thread Justin Mclean
Hi, > Thanks for your clarification. We have submitted a PR[1] to address this > issue. I believe this issue will be resolved quickly. > > [1] https://github.com/apache/incubator-streampark/pull/3689 That looks good to me, thanks. Kind Regards, Justin

Re: [DISCUSS] Drop the incubator- prefix for podling's GitHub repo

2024-04-29 Thread tison
Before starting the vote, I found (perhaps) a final question: Shall we thus guide all the new podlings to enter the incubator without incubator- prefix and thus converge all the current podlings are in form apache/foo? I'm afraid that if new podling can still have the incubator- prefix, it can

Re: [VOTE] Release Apache StreamPark(Incubating) 2.1.4-rc1

2024-04-29 Thread Huajie Wang
Thanks for your clarification. We have submitted a PR[1] to address this issue. I believe this issue will be resolved quickly. [1] https://github.com/apache/incubator-streampark/pull/3689 Best, Huajie Wang tison 于2024年4月29日周一 16:02写道: > > So a condition of including MIT licensed code is

Re: [VOTE] Release Apache StreamPark(Incubating) 2.1.4-rc1

2024-04-29 Thread Huajie Wang
> But I found a potential issue that it's possible StreamPark copied some code from flink-sql-gateway to streampark-flink-sql-gateway. Thanks for your review work, the "streampark-flink-sql-gateway" module is in the dev branch and is currently in the development phase, As you said, some of the

Re: [VOTE] Release Apache StreamPark(Incubating) 2.1.4-rc1

2024-04-29 Thread tison
> Even if they have changed a lot, that could still be an issue. Copying an earlier version of a file is still an issue that needs to be dealt with. Even copying 5% of something needs to be treated correctly. The files seem to have permissive licenses, so there is no category X licensing issue, at

Re: [VOTE] Release Apache StreamPark(Incubating) 2.1.4-rc1

2024-04-29 Thread Justin Mclean
Hi, > * Some false positive on testing assertions utils. Match rate < 50%, file > is small, and those utilities are trivial. Even when I go to the "origins", > they are lost or changed a lot. Clearly not the same origin. Even if they have changed a lot, that could still be an issue. Copying an

Re: [VOTE] Release Apache StreamPark(Incubating) 2.1.4-rc1

2024-04-29 Thread tison
But I found a potential issue that it's possible StreamPark copied some code from flink-sql-gateway to streampark-flink-sql-gateway. If that's the case, we should convey this info in the LICENSE file. But since Flink is also an ASF project, we don't need to convey another copy of ALv2 and no need

Re: [VOTE] Release Apache StreamPark(Incubating) 2.1.4-rc1

2024-04-29 Thread tison
> It looks like additional third-party code might also be in the release, but it is difficult to tell. Yeah. The result gives a lot of noise. I run SCANOSS locally, and don't find any other third-party code included: * Many of the reference to streamx is the original project before StreamPark

Re: [VOTE] Release Apache StreamPark(Incubating) 2.1.4-rc1

2024-04-29 Thread tison
> vue-vben-admin is under MIT[1] license, In the LICENST[2] file of > StreamPark, we listed which files are copied from vue-vben-admin The issue here is that, as MIT license writes: > The above copyright notice and this permission notice shall be included in all copies or substantial portions of

Re: [VOTE] Release Apache StreamPark(Incubating) 2.1.4-rc1

2024-04-29 Thread Huajie Wang
> So a condition of including MIT licensed code is to include the relevant MIT license text. That seems to be missing for for vue-vben-admin , as I can’t find it anywhere in the release vue-vben-admin is under MIT[1] license, In the LICENST[2] file of StreamPark, we listed which files are copied

Re: [VOTE] Release Apache StreamPark(Incubating) 2.1.4-rc1

2024-04-29 Thread tison
Here is a draft that participants in this thread can review: [1] [1] https://github.com/apache/incubator-streampark/pull/3689 Best, tison. tison 于2024年4月29日周一 16:01写道: > > So a condition of including MIT licensed code is to include the > relevant MIT license text. > > Confirmed this is the

Re: [VOTE] Release Apache StreamPark(Incubating) 2.1.4-rc1

2024-04-29 Thread tison
> So a condition of including MIT licensed code is to include the relevant MIT license text. Confirmed this is the case. I found the mention to vue-vben-admin is licensed under MIT at [1] but I don't find the LICENSE of vue-vben-admin bundled also. I agree that the LICENSE file at [2] seems to

Re: [VOTE] Release Apache StreamPark(Incubating) 2.1.4-rc1

2024-04-29 Thread Justin Mclean
Hi, -1 (binding) from me I checked: - incubating in name - signatures and hashes correct - disclaimer exists - LICENSE is missing some info on the MIT license - NOTICE looks fine - I didn't compile from source So a condition of including MIT licensed code is to include the relevant MIT license

Re: [VOTE] Release Apache StreamPark(Incubating) 2.1.4-rc1

2024-04-29 Thread Huajie Wang
> To verify the release here is supposed to verify the release candidate stored at the Apache distribution, right? Is it wrong to verify the code repo downloaded by git clone? When we are releasing a new version, we will create a git tag for the release. Here, "-b v2.1.4-rc1[1]" is to verify

Re: [VOTE] Release Apache StreamPark(Incubating) 2.1.4-rc1

2024-04-29 Thread shaokang lv
hi Suyan, Thank you for your careful review. The version by git clone is the same as the apache release candidate. To avoid ambiguity, we'll add a comment to the vote template later. Best, Shaokang Lv Suyan 于2024年4月28日周日 21:20写道: > +1 binding > Apache ID: suyanhanx > > I checked: > > [x]

Re: [DISCUSS] Drop the incubator- prefix for podling's GitHub repo

2024-04-29 Thread ConradJam
As a developer on the Apache Amoro project, I believe it's crucial to prominently display the project's status as an incubator, whether by attaching it to the project prefix or featuring it on the website. Most individuals typically recognize that a project is in incubation through the project's