You are viewing a plain text version of this content. The canonical link for it is here.
Posted to general@incubator.apache.org by Bertty Contreras <be...@scalytics.io> on 2021/10/23 11:52:20 UTC

[VOTE] Release Apache Wayang (Incubating) 0.6.0

Hello Incubator Community,

The Apache Wayang community has voted on and approved a proposal to release
Apache Wayang(Incubating) version v0.6.0-RC4.

We now kindly request the Incubator PMC members review and vote on this
incubator release.

Wayang community vote thread:
https://www.mail-archive.com/dev@wayang.apache.org/msg00446.html

Vote result thread:
https://www.mail-archive.com/dev@wayang.apache.org/msg00458.html

[ ] +1 Release this package as Apache Wayang v0.6.0
[ ] +0
[ ] -1 Do not release this package because ...

To learn more about Apache Wayang (Incubating), please see
https://wayang.apache.org/

Release notes:
https://github.com/apache/incubator-wayang/blob/rel/0.6.0/RELEASE_NOTES

The tag to be voted on is wayang-0.6.0 (commit 9fe19a1):
https://github.com/apache/incubator-wayang/tree/rel/0.6.0

The release files, including signatures, digests, etc. can be found at:
https://dist.apache.org/repos/dist/dev/incubator/wayang/0.6.0/rc4

Signatures used for Wayang RC's can be found in this file:
https://dist.apache.org/repos/dist/dev/incubator/wayang/KEYS

The staging repository for this release can be found at:
https://repository.apache.org/content/repositories/orgapachewayang-1005/org/apache/wayang

Build instructions:
https://github.com/apache/incubator-wayang/blob/rel/0.6.0/README.md#how-to-use-wayang

Thanks,
Bertty Contreras Rojas

AW: [VOTE] Release Apache Wayang (Incubating) 0.6.0

Posted by Christofer Dutz <ch...@c-ware.de>.
+1 binding

Carrying over my +1 from the vote on the project list

Chris


-----Ursprüngliche Nachricht-----
Von: Alexander Alten <al...@scalytics.io> 
Gesendet: Samstag, 23. Oktober 2021 14:44
An: general@incubator.apache.org
Betreff: Re: [VOTE] Release Apache Wayang (Incubating) 0.6.0

+1 (binding)

--alex

On Sat, Oct 23, 2021, 13:52 Bertty Contreras <be...@scalytics.io> wrote:

> Hello Incubator Community,
>
> The Apache Wayang community has voted on and approved a proposal to 
> release Apache Wayang(Incubating) version v0.6.0-RC4.
>
> We now kindly request the Incubator PMC members review and vote on 
> this incubator release.
>
> Wayang community vote thread:
> https://www.mail-archive.com/dev@wayang.apache.org/msg00446.html
>
> Vote result thread:
> https://www.mail-archive.com/dev@wayang.apache.org/msg00458.html
>
> [ ] +1 Release this package as Apache Wayang v0.6.0 [ ] +0 [ ] -1 Do 
> not release this package because ...
>
> To learn more about Apache Wayang (Incubating), please see 
> https://wayang.apache.org/
>
> Release notes:
> https://github.com/apache/incubator-wayang/blob/rel/0.6.0/RELEASE_NOTE
> S
>
> The tag to be voted on is wayang-0.6.0 (commit 9fe19a1):
> https://github.com/apache/incubator-wayang/tree/rel/0.6.0
>
> The release files, including signatures, digests, etc. can be found at:
> https://dist.apache.org/repos/dist/dev/incubator/wayang/0.6.0/rc4
>
> Signatures used for Wayang RC's can be found in this file:
> https://dist.apache.org/repos/dist/dev/incubator/wayang/KEYS
>
> The staging repository for this release can be found at:
>
> https://repository.apache.org/content/repositories/orgapachewayang-100
> 5/org/apache/wayang
>
> Build instructions:
>
> https://github.com/apache/incubator-wayang/blob/rel/0.6.0/README.md#ho
> w-to-use-wayang
>
> Thanks,
> Bertty Contreras Rojas
>

---------------------------------------------------------------------
To unsubscribe, e-mail: general-unsubscribe@incubator.apache.org
For additional commands, e-mail: general-help@incubator.apache.org

Re: [VOTE] Release Apache Wayang (Incubating) 0.6.0

Posted by Alexander Alten <al...@scalytics.io>.
+1 (binding)

--alex

On Sat, Oct 23, 2021, 13:52 Bertty Contreras <be...@scalytics.io> wrote:

> Hello Incubator Community,
>
> The Apache Wayang community has voted on and approved a proposal to release
> Apache Wayang(Incubating) version v0.6.0-RC4.
>
> We now kindly request the Incubator PMC members review and vote on this
> incubator release.
>
> Wayang community vote thread:
> https://www.mail-archive.com/dev@wayang.apache.org/msg00446.html
>
> Vote result thread:
> https://www.mail-archive.com/dev@wayang.apache.org/msg00458.html
>
> [ ] +1 Release this package as Apache Wayang v0.6.0
> [ ] +0
> [ ] -1 Do not release this package because ...
>
> To learn more about Apache Wayang (Incubating), please see
> https://wayang.apache.org/
>
> Release notes:
> https://github.com/apache/incubator-wayang/blob/rel/0.6.0/RELEASE_NOTES
>
> The tag to be voted on is wayang-0.6.0 (commit 9fe19a1):
> https://github.com/apache/incubator-wayang/tree/rel/0.6.0
>
> The release files, including signatures, digests, etc. can be found at:
> https://dist.apache.org/repos/dist/dev/incubator/wayang/0.6.0/rc4
>
> Signatures used for Wayang RC's can be found in this file:
> https://dist.apache.org/repos/dist/dev/incubator/wayang/KEYS
>
> The staging repository for this release can be found at:
>
> https://repository.apache.org/content/repositories/orgapachewayang-1005/org/apache/wayang
>
> Build instructions:
>
> https://github.com/apache/incubator-wayang/blob/rel/0.6.0/README.md#how-to-use-wayang
>
> Thanks,
> Bertty Contreras Rojas
>

Re: [VOTE] Release Apache Wayang (Incubating) 0.6.0

Posted by Justin Mclean <ju...@classsoftware.com>.
Hi,

> Graphchi: Graphchi has an Apache License. The authors didn't mention the
> license in the repository of graphchi-java(it is a wrapper of
> Graphchi-cpp).

Just because Graphchi-cpp is Apache licensed doesn’t mean graphchi-java is or that people understand that they are contributing code under that licence. I’d reach out to teh project and ask them to add a license to their repository.

> PageRank: we created an operator called PageRankOperator [9]; you can see
> the use of WayangPlans#pageRankWithDictionaryCompression in [10]. However,
> the PageRank will be converted depending on the optimiser's decision to one
> of the implementations [11, 12,13, 14]. In the different implementations,
> we implement the page rank algorithm following the API of the platform, or
> we use an implementation of the platform. Does this
> explanation address your question?

Not really. It still looks like code has been copied form that page which is not under a compatible license. Copying code from Wikipedia or Stack Overflow can be problematic.

> NOTICE: we include Apache Spark, Apache Flink, Apache Hadoop, Apache Giraph
> and Graphchi-Java, because they contain dependencies that are transitive to
> the Wayang code

Only things that are included in the source release should be mentioned in NOTICE or LICENSE. Dependancies, especially transitive ones do not need to be mentioned. See [1]


> and when wayang review the dependencies licenses [15], the
> projects mentioned and their dependencies are excluded [16]

Sorry I’m not sure what you mean by this.

Thanks,
Justin

1. https://infra.apache.org/licensing-howto.html#guiding
---------------------------------------------------------------------
To unsubscribe, e-mail: general-unsubscribe@incubator.apache.org
For additional commands, e-mail: general-help@incubator.apache.org


Re: [VOTE] Release Apache Wayang (Incubating) 0.6.0

Posted by bertty contreras <be...@gmail.com>.
Hi Justin,

Thanks you for you feedback, I write some answer to your question :D

Graphchi: Graphchi has an Apache License. The authors didn't mention the
license in the repository of graphchi-java(it is a wrapper of
Graphchi-cpp). Still, in the graphchi-cpp you can find the license[1];
also, it is possible to see the reference to the licence of graphchi-java
in the maven repository [2].

JSON: The class JSONArray [3] and JSONObject [4] have the same name that
you can find in [5] and [6] in the project JSON-java [7]. We implemented
the wrapper with the same name and signature because of compatible licenses
issues and several code dependencies. The wrapper was created not to affect
the functionality of the current code but to be enabled to replace the
library JSON-java for Jackson[8]. Jackson has an Apache License.

PageRank: we created an operator called PageRankOperator [9]; you can see
the use of WayangPlans#pageRankWithDictionaryCompression in [10]. However,
the PageRank will be converted depending on the optimiser's decision to one
of the implementations [11, 12,13, 14]. In the different implementations,
we implement the page rank algorithm following the API of the platform, or
we use an implementation of the platform. Does this
explanation address your question?

NOTICE: we include Apache Spark, Apache Flink, Apache Hadoop, Apache Giraph
and Graphchi-Java, because they contain dependencies that are transitive to
the Wayang code, and when wayang review the dependencies licenses [15], the
projects mentioned and their dependencies are excluded [16]

Copyright: we will modify them as soon as possible.

Best regards,
Bertty

[1] https://github.com/GraphChi/graphchi-cpp#license
[2] https://mvnrepository.com/artifact/org.graphchi/graphchi-java
[3]
https://github.com/apache/incubator-wayang/blob/main/wayang-commons/wayang-core/src/main/java/org/apache/wayang/core/util/json/JSONArray.java
[4]
https://github.com/apache/incubator-wayang/blob/main/wayang-commons/wayang-core/src/main/java/org/apache/wayang/core/util/json/JSONObject.java
[5]
https://github.com/stleary/JSON-java/blob/master/src/main/java/org/json/JSONArray.java
[6]
https://github.com/stleary/JSON-java/blob/master/src/main/java/org/json/JSONObject.java
[7] https://github.com/stleary/JSON-java
[8] https://github.com/FasterXML/jackson-core
[9]
https://github.com/apache/incubator-wayang/blob/main/wayang-commons/wayang-basic/src/main/java/org/apache/wayang/basic/operators/PageRankOperator.java
[10]
https://github.com/apache/incubator-wayang/blob/main/wayang-tests-integration/code/test/java/org/apache/wayang/tests/WayangPlans.java#L665-L668
[11]
https://github.com/apache/incubator-wayang/blob/main/wayang-platforms/wayang-spark/code/main/scala/org/apache/wayang/spark/operators/graph/SparkPageRankOperator.scala
[12]
https://github.com/apache/incubator-wayang/blob/main/wayang-platforms/wayang-flink/code/main/java/org/apache/wayang/flink/operators/FlinkPageRankOperator.java
[13]
https://github.com/apache/incubator-wayang/blob/main/wayang-platforms/wayang-giraph/src/main/java/org/apache/wayang/giraph/operators/GiraphPageRankOperator.java
[14]
https://github.com/apache/incubator-wayang/blob/main/wayang-platforms/wayang-graphchi/src/main/java/org/apache/wayang/graphchi/operators/GraphChiPageRankOperator.java
[15]
https://github.com/apache/incubator-wayang/blob/main/pom.xml#L1248-L1279
[16]
https://github.com/apache/incubator-wayang/blob/main/pom.xml#L1274-L1276


El dom, 24 oct 2021 a las 3:06, Justin Mclean (<ju...@classsoftware.com>)
escribió:

> Hi,
>
> -1 (binding) as this release seems to contain 3rd party code incorrectly
> labeled as ASF code and contains code of unknown and category X licenses.
> But it’s not 100% clear in all cases. You might want to consider using the
> work in progress disclaimer while you fix these things.
>
> I checked:
> - incubating in name
> - signatures and hashes are fine
> - LICENSE seems to be missing 3rd party licensed code. Information about
> licenses goes in LICENSE not NOTICE. Looks like this release may include
> creative commons licensed code [2][3]? Missing license for this 3rd party
> code [4][5]?
> - NOTICE seems malformed. I don’t know why it mentions these projects or
> why it names them in the way it does. Is any of this software included in
> the release? if so where?
> - Looks like release may contain code of an unknown license [1]
> - Source files have ASF headers. However it look likes some 3rd party code
> (graphchi) incorrectly has had ASF headers added?
> - Some copyright line are incorrect "Copyright 2020 Apache (incubating)
> Wayang Team” and "Copyright 2020, 2021 Apache (incubating) Wayang Team”
> - No unexpected binary files
> - Can compile from source
>
> Kind Regards,
> Justin
>
> 1. https://github.com/GraphChi/graphchi-java
> 2. https://en.wikipedia.org/wiki/PageRank
> 3.
> ./wayang-tests-integration/code/test/java/org/apache/wayang/tests/WayangPlans.java
>
> 4../wayang-commons/wayang-core/src/main/java/org/apache/wayang/core/util/json/JSONArray.java
> 5.
> ./wayang-commons/wayang-core/src/main/java/org/apache/wayang/core/util/json/JSONObject.java
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: general-unsubscribe@incubator.apache.org
> For additional commands, e-mail: general-help@incubator.apache.org
>
>

Re: [VOTE] Release Apache Wayang (Incubating) 0.6.0

Posted by Justin Mclean <ju...@classsoftware.com>.
Hi,

-1 (binding) as this release seems to contain 3rd party code incorrectly labeled as ASF code and contains code of unknown and category X licenses. But it’s not 100% clear in all cases. You might want to consider using the work in progress disclaimer while you fix these things.

I checked:
- incubating in name
- signatures and hashes are fine
- LICENSE seems to be missing 3rd party licensed code. Information about licenses goes in LICENSE not NOTICE. Looks like this release may include creative commons licensed code [2][3]? Missing license for this 3rd party code [4][5]?
- NOTICE seems malformed. I don’t know why it mentions these projects or why it names them in the way it does. Is any of this software included in the release? if so where?
- Looks like release may contain code of an unknown license [1]
- Source files have ASF headers. However it look likes some 3rd party code (graphchi) incorrectly has had ASF headers added?
- Some copyright line are incorrect "Copyright 2020 Apache (incubating) Wayang Team” and "Copyright 2020, 2021 Apache (incubating) Wayang Team”
- No unexpected binary files
- Can compile from source

Kind Regards,
Justin

1. https://github.com/GraphChi/graphchi-java
2. https://en.wikipedia.org/wiki/PageRank
3. ./wayang-tests-integration/code/test/java/org/apache/wayang/tests/WayangPlans.java
4../wayang-commons/wayang-core/src/main/java/org/apache/wayang/core/util/json/JSONArray.java
5. ./wayang-commons/wayang-core/src/main/java/org/apache/wayang/core/util/json/JSONObject.java
---------------------------------------------------------------------
To unsubscribe, e-mail: general-unsubscribe@incubator.apache.org
For additional commands, e-mail: general-help@incubator.apache.org


Re: [VOTE] Release Apache Wayang (Incubating) 0.6.0

Posted by sebb <se...@gmail.com>.
On Sat, 23 Oct 2021 at 16:20, bertty contreras
<be...@gmail.com> wrote:
>
> Hi Sebb,
>
> The directory was created :D
>
> The KEYS file got updated at [1] and synced on [2]. From now we will use
> just the canonical link [2] for the KEYS file

Excellent!
That was quick.

Remember that the same file is used for archived releases, so an entry
should never be removed
if it has been used to sign a release

> Best regards,
> Bertty
>
> [1]https://dist.apache.org/repos/dist/release/incubator/wayang
> [2]http://downloads.apache.org/incubator/wayang/KEYS
>
> El sáb, 23 oct 2021 a las 16:58, sebb (<se...@gmail.com>) escribió:
>
> > On Sat, 23 Oct 2021 at 12:52, Bertty Contreras <be...@scalytics.io>
> > wrote:
> > >
> > > Hello Incubator Community,
> > >
> > > The Apache Wayang community has voted on and approved a proposal to
> > release
> > > Apache Wayang(Incubating) version v0.6.0-RC4.
> > >
> > > We now kindly request the Incubator PMC members review and vote on this
> > > incubator release.
> > >
> > > Wayang community vote thread:
> > > https://www.mail-archive.com/dev@wayang.apache.org/msg00446.html
> > >
> > > Vote result thread:
> > > https://www.mail-archive.com/dev@wayang.apache.org/msg00458.html
> > >
> > > [ ] +1 Release this package as Apache Wayang v0.6.0
> > > [ ] +0
> > > [ ] -1 Do not release this package because ...
> > >
> > > To learn more about Apache Wayang (Incubating), please see
> > > https://wayang.apache.org/
> > >
> > > Release notes:
> > > https://github.com/apache/incubator-wayang/blob/rel/0.6.0/RELEASE_NOTES
> > >
> > > The tag to be voted on is wayang-0.6.0 (commit 9fe19a1):
> > > https://github.com/apache/incubator-wayang/tree/rel/0.6.0
> > >
> > > The release files, including signatures, digests, etc. can be found at:
> > > https://dist.apache.org/repos/dist/dev/incubator/wayang/0.6.0/rc4
> > >
> > > Signatures used for Wayang RC's can be found in this file:
> > > https://dist.apache.org/repos/dist/dev/incubator/wayang/KEYS
> >
> > In future, please always link the KEYS file from
> > http://downloads.apache.org/incubator/wayang/KEYS
> > For reasons, please see the recent Kyuubi and Shenyu votes.
> >
> > In your case, there is no dist/release directory yet.
> > This needs to be set up before a release can be made:
> >
> > https://dist.apache.org/repos/dist/release/incubator/wayang
> >
> > Ask one of your mentors to get this done.
> >
> > > The staging repository for this release can be found at:
> > >
> > https://repository.apache.org/content/repositories/orgapachewayang-1005/org/apache/wayang
> > >
> > > Build instructions:
> > >
> > https://github.com/apache/incubator-wayang/blob/rel/0.6.0/README.md#how-to-use-wayang
> > >
> > > Thanks,
> > > Bertty Contreras Rojas
> >
> > ---------------------------------------------------------------------
> > To unsubscribe, e-mail: general-unsubscribe@incubator.apache.org
> > For additional commands, e-mail: general-help@incubator.apache.org
> >
> >

---------------------------------------------------------------------
To unsubscribe, e-mail: general-unsubscribe@incubator.apache.org
For additional commands, e-mail: general-help@incubator.apache.org


Re: [VOTE] Release Apache Wayang (Incubating) 0.6.0

Posted by bertty contreras <be...@gmail.com>.
Hi Sebb,

The directory was created :D

The KEYS file got updated at [1] and synced on [2]. From now we will use
just the canonical link [2] for the KEYS file

Best regards,
Bertty

[1]https://dist.apache.org/repos/dist/release/incubator/wayang
[2]http://downloads.apache.org/incubator/wayang/KEYS

El sáb, 23 oct 2021 a las 16:58, sebb (<se...@gmail.com>) escribió:

> On Sat, 23 Oct 2021 at 12:52, Bertty Contreras <be...@scalytics.io>
> wrote:
> >
> > Hello Incubator Community,
> >
> > The Apache Wayang community has voted on and approved a proposal to
> release
> > Apache Wayang(Incubating) version v0.6.0-RC4.
> >
> > We now kindly request the Incubator PMC members review and vote on this
> > incubator release.
> >
> > Wayang community vote thread:
> > https://www.mail-archive.com/dev@wayang.apache.org/msg00446.html
> >
> > Vote result thread:
> > https://www.mail-archive.com/dev@wayang.apache.org/msg00458.html
> >
> > [ ] +1 Release this package as Apache Wayang v0.6.0
> > [ ] +0
> > [ ] -1 Do not release this package because ...
> >
> > To learn more about Apache Wayang (Incubating), please see
> > https://wayang.apache.org/
> >
> > Release notes:
> > https://github.com/apache/incubator-wayang/blob/rel/0.6.0/RELEASE_NOTES
> >
> > The tag to be voted on is wayang-0.6.0 (commit 9fe19a1):
> > https://github.com/apache/incubator-wayang/tree/rel/0.6.0
> >
> > The release files, including signatures, digests, etc. can be found at:
> > https://dist.apache.org/repos/dist/dev/incubator/wayang/0.6.0/rc4
> >
> > Signatures used for Wayang RC's can be found in this file:
> > https://dist.apache.org/repos/dist/dev/incubator/wayang/KEYS
>
> In future, please always link the KEYS file from
> http://downloads.apache.org/incubator/wayang/KEYS
> For reasons, please see the recent Kyuubi and Shenyu votes.
>
> In your case, there is no dist/release directory yet.
> This needs to be set up before a release can be made:
>
> https://dist.apache.org/repos/dist/release/incubator/wayang
>
> Ask one of your mentors to get this done.
>
> > The staging repository for this release can be found at:
> >
> https://repository.apache.org/content/repositories/orgapachewayang-1005/org/apache/wayang
> >
> > Build instructions:
> >
> https://github.com/apache/incubator-wayang/blob/rel/0.6.0/README.md#how-to-use-wayang
> >
> > Thanks,
> > Bertty Contreras Rojas
>
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: general-unsubscribe@incubator.apache.org
> For additional commands, e-mail: general-help@incubator.apache.org
>
>

Re: [VOTE] Release Apache Wayang (Incubating) 0.6.0

Posted by sebb <se...@gmail.com>.
On Sat, 23 Oct 2021 at 12:52, Bertty Contreras <be...@scalytics.io> wrote:
>
> Hello Incubator Community,
>
> The Apache Wayang community has voted on and approved a proposal to release
> Apache Wayang(Incubating) version v0.6.0-RC4.
>
> We now kindly request the Incubator PMC members review and vote on this
> incubator release.
>
> Wayang community vote thread:
> https://www.mail-archive.com/dev@wayang.apache.org/msg00446.html
>
> Vote result thread:
> https://www.mail-archive.com/dev@wayang.apache.org/msg00458.html
>
> [ ] +1 Release this package as Apache Wayang v0.6.0
> [ ] +0
> [ ] -1 Do not release this package because ...
>
> To learn more about Apache Wayang (Incubating), please see
> https://wayang.apache.org/
>
> Release notes:
> https://github.com/apache/incubator-wayang/blob/rel/0.6.0/RELEASE_NOTES
>
> The tag to be voted on is wayang-0.6.0 (commit 9fe19a1):
> https://github.com/apache/incubator-wayang/tree/rel/0.6.0
>
> The release files, including signatures, digests, etc. can be found at:
> https://dist.apache.org/repos/dist/dev/incubator/wayang/0.6.0/rc4
>
> Signatures used for Wayang RC's can be found in this file:
> https://dist.apache.org/repos/dist/dev/incubator/wayang/KEYS

In future, please always link the KEYS file from
http://downloads.apache.org/incubator/wayang/KEYS
For reasons, please see the recent Kyuubi and Shenyu votes.

In your case, there is no dist/release directory yet.
This needs to be set up before a release can be made:

https://dist.apache.org/repos/dist/release/incubator/wayang

Ask one of your mentors to get this done.

> The staging repository for this release can be found at:
> https://repository.apache.org/content/repositories/orgapachewayang-1005/org/apache/wayang
>
> Build instructions:
> https://github.com/apache/incubator-wayang/blob/rel/0.6.0/README.md#how-to-use-wayang
>
> Thanks,
> Bertty Contreras Rojas

---------------------------------------------------------------------
To unsubscribe, e-mail: general-unsubscribe@incubator.apache.org
For additional commands, e-mail: general-help@incubator.apache.org