You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@wayang.apache.org by Bertty Contreras <be...@scalytics.io> on 2021/10/27 12:57:48 UTC
PRE-RELEASE
Hello Folks,
I did the modification on the code mentioned by Justin.
Please double check the list and let me know if something is missing or
wrong.
After you check I will perform RC5.
All the code is on https://github.com/apache/incubator-wayang in the main
branch :D
Best regards,
Bertty
Re: PRE-RELEASE
Posted by Rodrigo Pardo Meza <ro...@gmail.com>.
Hi Mentors,
Now, after discussing with Justin, the page rank issue is solved.
Nevertheless, he insists that the Wayang Notice file has issues.
He said that the Notice must only list things that come from a Software
grant, where a copyright header was charged or from other Notice files.
We are not pretty sure about how to proceed to fix this. Could you explain
it to us better? He said we need to direct this kind of question to you
guys.
Thanks in advance.
Best regards
El jue, 28 oct 2021 a las 19:29, Bertty Contreras (<be...@scalytics.io>)
escribió:
> Nice to know, I will write to him :D and explain what we did :D
>
> On Thu, Oct 28, 2021 at 7:09 PM Christofer Dutz <christofer.dutz@c-ware.de
> >
> wrote:
>
> > Just ping him in Slack .. he's usually hanging around there ;-)
> >
> > He lives in Sydney (Australia) so ... just that you know when you can
> > expect him to be online.
> >
> > Chris
> >
> >
> > -----Ursprüngliche Nachricht-----
> > Von: Bertty Contreras <be...@scalytics.io>
> > Gesendet: Donnerstag, 28. Oktober 2021 19:07
> > An: dev@wayang.apache.org
> > Betreff: Re: PRE-RELEASE
> >
> > Hi Chris,
> >
> > Do we send an email directly to him or how do we ask him ?
> >
> > Best regards,
> > Bertty
> >
> > On Thu, Oct 28, 2021 at 5:43 PM Rodrigo Pardo Meza <
> > ro.pardo.meza@gmail.com>
> > wrote:
> >
> > > Hello,
> > >
> > > That would be great; after the changes the notice seems to be ok plus
> > > the page rank example has been updated avoiding the issue.
> > >
> > > Seems fine to me.
> > >
> > > Best regards
> > >
> > > El jue, 28 oct 2021 a las 17:11, Christofer Dutz (<
> > > christofer.dutz@c-ware.de>)
> > > escribió:
> > >
> > > > Perhaps we should ask Justin to have a look if his issues have been
> > > > dealt with.
> > > >
> > > > Chris
> > > >
> > > >
> > > > -----Ursprüngliche Nachricht-----
> > > > Von: Alexander Alten <al...@scalytics.io>
> > > > Gesendet: Donnerstag, 28. Oktober 2021 16:37
> > > > An: dev@wayang.apache.org
> > > > Betreff: Re: PRE-RELEASE
> > > >
> > > > Nope, does not looking good. Build passes only when PR35 is merged -
> > > > a NOTICE does not have to have .md - that causes license issues with
> > rat.
> > > >
> > > > Local branch (with merged 35) build successful, both Java8 and 11.
> > > > Since we removed PageRank from Wikipedia I think we are good after
> the
> > merge.
> > > >
> > > > Cheers,
> > > > —Alex
> > > >
> > > > > On 28. Oct 2021, at 15:22, jorge Arnulfo Quiané Ruiz <
> > > > quianeufo@gmail.com> wrote:
> > > > >
> > > > > Hello guys,
> > > > >
> > > > > I checked the list and everything looks good from my side. I am
> > > > > thus for starting the RC5.
> > > > >
> > > > > Thanks for the great job team!
> > > > > Jorge
> > > > >
> > > > > On Wed, Oct 27, 2021 at 2:58 PM Bertty Contreras
> > > > > <be...@scalytics.io>
> > > > > wrote:
> > > > >
> > > > >> Hello Folks,
> > > > >>
> > > > >> I did the modification on the code mentioned by Justin.
> > > > >>
> > > > >> Please double check the list and let me know if something is
> > > > >> missing or wrong.
> > > > >>
> > > > >> After you check I will perform RC5.
> > > > >>
> > > > >> All the code is on https://github.com/apache/incubator-wayang in
> > > > >> the main branch :D
> > > > >>
> > > > >> Best regards,
> > > > >> Bertty
> > > > >>
> > > >
> > > >
> > >
> >
>
Re: PRE-RELEASE
Posted by Bertty Contreras <be...@scalytics.io>.
Nice to know, I will write to him :D and explain what we did :D
On Thu, Oct 28, 2021 at 7:09 PM Christofer Dutz <ch...@c-ware.de>
wrote:
> Just ping him in Slack .. he's usually hanging around there ;-)
>
> He lives in Sydney (Australia) so ... just that you know when you can
> expect him to be online.
>
> Chris
>
>
> -----Ursprüngliche Nachricht-----
> Von: Bertty Contreras <be...@scalytics.io>
> Gesendet: Donnerstag, 28. Oktober 2021 19:07
> An: dev@wayang.apache.org
> Betreff: Re: PRE-RELEASE
>
> Hi Chris,
>
> Do we send an email directly to him or how do we ask him ?
>
> Best regards,
> Bertty
>
> On Thu, Oct 28, 2021 at 5:43 PM Rodrigo Pardo Meza <
> ro.pardo.meza@gmail.com>
> wrote:
>
> > Hello,
> >
> > That would be great; after the changes the notice seems to be ok plus
> > the page rank example has been updated avoiding the issue.
> >
> > Seems fine to me.
> >
> > Best regards
> >
> > El jue, 28 oct 2021 a las 17:11, Christofer Dutz (<
> > christofer.dutz@c-ware.de>)
> > escribió:
> >
> > > Perhaps we should ask Justin to have a look if his issues have been
> > > dealt with.
> > >
> > > Chris
> > >
> > >
> > > -----Ursprüngliche Nachricht-----
> > > Von: Alexander Alten <al...@scalytics.io>
> > > Gesendet: Donnerstag, 28. Oktober 2021 16:37
> > > An: dev@wayang.apache.org
> > > Betreff: Re: PRE-RELEASE
> > >
> > > Nope, does not looking good. Build passes only when PR35 is merged -
> > > a NOTICE does not have to have .md - that causes license issues with
> rat.
> > >
> > > Local branch (with merged 35) build successful, both Java8 and 11.
> > > Since we removed PageRank from Wikipedia I think we are good after the
> merge.
> > >
> > > Cheers,
> > > —Alex
> > >
> > > > On 28. Oct 2021, at 15:22, jorge Arnulfo Quiané Ruiz <
> > > quianeufo@gmail.com> wrote:
> > > >
> > > > Hello guys,
> > > >
> > > > I checked the list and everything looks good from my side. I am
> > > > thus for starting the RC5.
> > > >
> > > > Thanks for the great job team!
> > > > Jorge
> > > >
> > > > On Wed, Oct 27, 2021 at 2:58 PM Bertty Contreras
> > > > <be...@scalytics.io>
> > > > wrote:
> > > >
> > > >> Hello Folks,
> > > >>
> > > >> I did the modification on the code mentioned by Justin.
> > > >>
> > > >> Please double check the list and let me know if something is
> > > >> missing or wrong.
> > > >>
> > > >> After you check I will perform RC5.
> > > >>
> > > >> All the code is on https://github.com/apache/incubator-wayang in
> > > >> the main branch :D
> > > >>
> > > >> Best regards,
> > > >> Bertty
> > > >>
> > >
> > >
> >
>
AW: PRE-RELEASE
Posted by Christofer Dutz <ch...@c-ware.de>.
Just ping him in Slack .. he's usually hanging around there ;-)
He lives in Sydney (Australia) so ... just that you know when you can expect him to be online.
Chris
-----Ursprüngliche Nachricht-----
Von: Bertty Contreras <be...@scalytics.io>
Gesendet: Donnerstag, 28. Oktober 2021 19:07
An: dev@wayang.apache.org
Betreff: Re: PRE-RELEASE
Hi Chris,
Do we send an email directly to him or how do we ask him ?
Best regards,
Bertty
On Thu, Oct 28, 2021 at 5:43 PM Rodrigo Pardo Meza <ro...@gmail.com>
wrote:
> Hello,
>
> That would be great; after the changes the notice seems to be ok plus
> the page rank example has been updated avoiding the issue.
>
> Seems fine to me.
>
> Best regards
>
> El jue, 28 oct 2021 a las 17:11, Christofer Dutz (<
> christofer.dutz@c-ware.de>)
> escribió:
>
> > Perhaps we should ask Justin to have a look if his issues have been
> > dealt with.
> >
> > Chris
> >
> >
> > -----Ursprüngliche Nachricht-----
> > Von: Alexander Alten <al...@scalytics.io>
> > Gesendet: Donnerstag, 28. Oktober 2021 16:37
> > An: dev@wayang.apache.org
> > Betreff: Re: PRE-RELEASE
> >
> > Nope, does not looking good. Build passes only when PR35 is merged -
> > a NOTICE does not have to have .md - that causes license issues with rat.
> >
> > Local branch (with merged 35) build successful, both Java8 and 11.
> > Since we removed PageRank from Wikipedia I think we are good after the merge.
> >
> > Cheers,
> > —Alex
> >
> > > On 28. Oct 2021, at 15:22, jorge Arnulfo Quiané Ruiz <
> > quianeufo@gmail.com> wrote:
> > >
> > > Hello guys,
> > >
> > > I checked the list and everything looks good from my side. I am
> > > thus for starting the RC5.
> > >
> > > Thanks for the great job team!
> > > Jorge
> > >
> > > On Wed, Oct 27, 2021 at 2:58 PM Bertty Contreras
> > > <be...@scalytics.io>
> > > wrote:
> > >
> > >> Hello Folks,
> > >>
> > >> I did the modification on the code mentioned by Justin.
> > >>
> > >> Please double check the list and let me know if something is
> > >> missing or wrong.
> > >>
> > >> After you check I will perform RC5.
> > >>
> > >> All the code is on https://github.com/apache/incubator-wayang in
> > >> the main branch :D
> > >>
> > >> Best regards,
> > >> Bertty
> > >>
> >
> >
>
Re: PRE-RELEASE
Posted by Bertty Contreras <be...@scalytics.io>.
Hi Chris,
Do we send an email directly to him or how do we ask him ?
Best regards,
Bertty
On Thu, Oct 28, 2021 at 5:43 PM Rodrigo Pardo Meza <ro...@gmail.com>
wrote:
> Hello,
>
> That would be great; after the changes the notice seems to be ok plus the
> page rank example has been updated avoiding the issue.
>
> Seems fine to me.
>
> Best regards
>
> El jue, 28 oct 2021 a las 17:11, Christofer Dutz (<
> christofer.dutz@c-ware.de>)
> escribió:
>
> > Perhaps we should ask Justin to have a look if his issues have been dealt
> > with.
> >
> > Chris
> >
> >
> > -----Ursprüngliche Nachricht-----
> > Von: Alexander Alten <al...@scalytics.io>
> > Gesendet: Donnerstag, 28. Oktober 2021 16:37
> > An: dev@wayang.apache.org
> > Betreff: Re: PRE-RELEASE
> >
> > Nope, does not looking good. Build passes only when PR35 is merged - a
> > NOTICE does not have to have .md - that causes license issues with rat.
> >
> > Local branch (with merged 35) build successful, both Java8 and 11. Since
> > we removed PageRank from Wikipedia I think we are good after the merge.
> >
> > Cheers,
> > —Alex
> >
> > > On 28. Oct 2021, at 15:22, jorge Arnulfo Quiané Ruiz <
> > quianeufo@gmail.com> wrote:
> > >
> > > Hello guys,
> > >
> > > I checked the list and everything looks good from my side. I am thus
> > > for starting the RC5.
> > >
> > > Thanks for the great job team!
> > > Jorge
> > >
> > > On Wed, Oct 27, 2021 at 2:58 PM Bertty Contreras <be...@scalytics.io>
> > > wrote:
> > >
> > >> Hello Folks,
> > >>
> > >> I did the modification on the code mentioned by Justin.
> > >>
> > >> Please double check the list and let me know if something is missing
> > >> or wrong.
> > >>
> > >> After you check I will perform RC5.
> > >>
> > >> All the code is on https://github.com/apache/incubator-wayang in the
> > >> main branch :D
> > >>
> > >> Best regards,
> > >> Bertty
> > >>
> >
> >
>
Re: PRE-RELEASE
Posted by Rodrigo Pardo Meza <ro...@gmail.com>.
Hello,
That would be great; after the changes the notice seems to be ok plus the
page rank example has been updated avoiding the issue.
Seems fine to me.
Best regards
El jue, 28 oct 2021 a las 17:11, Christofer Dutz (<ch...@c-ware.de>)
escribió:
> Perhaps we should ask Justin to have a look if his issues have been dealt
> with.
>
> Chris
>
>
> -----Ursprüngliche Nachricht-----
> Von: Alexander Alten <al...@scalytics.io>
> Gesendet: Donnerstag, 28. Oktober 2021 16:37
> An: dev@wayang.apache.org
> Betreff: Re: PRE-RELEASE
>
> Nope, does not looking good. Build passes only when PR35 is merged - a
> NOTICE does not have to have .md - that causes license issues with rat.
>
> Local branch (with merged 35) build successful, both Java8 and 11. Since
> we removed PageRank from Wikipedia I think we are good after the merge.
>
> Cheers,
> —Alex
>
> > On 28. Oct 2021, at 15:22, jorge Arnulfo Quiané Ruiz <
> quianeufo@gmail.com> wrote:
> >
> > Hello guys,
> >
> > I checked the list and everything looks good from my side. I am thus
> > for starting the RC5.
> >
> > Thanks for the great job team!
> > Jorge
> >
> > On Wed, Oct 27, 2021 at 2:58 PM Bertty Contreras <be...@scalytics.io>
> > wrote:
> >
> >> Hello Folks,
> >>
> >> I did the modification on the code mentioned by Justin.
> >>
> >> Please double check the list and let me know if something is missing
> >> or wrong.
> >>
> >> After you check I will perform RC5.
> >>
> >> All the code is on https://github.com/apache/incubator-wayang in the
> >> main branch :D
> >>
> >> Best regards,
> >> Bertty
> >>
>
>
AW: PRE-RELEASE
Posted by Christofer Dutz <ch...@c-ware.de>.
Perhaps we should ask Justin to have a look if his issues have been dealt with.
Chris
-----Ursprüngliche Nachricht-----
Von: Alexander Alten <al...@scalytics.io>
Gesendet: Donnerstag, 28. Oktober 2021 16:37
An: dev@wayang.apache.org
Betreff: Re: PRE-RELEASE
Nope, does not looking good. Build passes only when PR35 is merged - a NOTICE does not have to have .md - that causes license issues with rat.
Local branch (with merged 35) build successful, both Java8 and 11. Since we removed PageRank from Wikipedia I think we are good after the merge.
Cheers,
—Alex
> On 28. Oct 2021, at 15:22, jorge Arnulfo Quiané Ruiz <qu...@gmail.com> wrote:
>
> Hello guys,
>
> I checked the list and everything looks good from my side. I am thus
> for starting the RC5.
>
> Thanks for the great job team!
> Jorge
>
> On Wed, Oct 27, 2021 at 2:58 PM Bertty Contreras <be...@scalytics.io>
> wrote:
>
>> Hello Folks,
>>
>> I did the modification on the code mentioned by Justin.
>>
>> Please double check the list and let me know if something is missing
>> or wrong.
>>
>> After you check I will perform RC5.
>>
>> All the code is on https://github.com/apache/incubator-wayang in the
>> main branch :D
>>
>> Best regards,
>> Bertty
>>
Re: PRE-RELEASE
Posted by Alexander Alten <al...@scalytics.io>.
Nope, does not looking good. Build passes only when PR35 is merged - a NOTICE does not have to have .md - that causes license issues with rat.
Local branch (with merged 35) build successful, both Java8 and 11. Since we removed PageRank from Wikipedia I think we are good after the merge.
Cheers,
—Alex
> On 28. Oct 2021, at 15:22, jorge Arnulfo Quiané Ruiz <qu...@gmail.com> wrote:
>
> Hello guys,
>
> I checked the list and everything looks good from my side. I am thus for
> starting the RC5.
>
> Thanks for the great job team!
> Jorge
>
> On Wed, Oct 27, 2021 at 2:58 PM Bertty Contreras <be...@scalytics.io>
> wrote:
>
>> Hello Folks,
>>
>> I did the modification on the code mentioned by Justin.
>>
>> Please double check the list and let me know if something is missing or
>> wrong.
>>
>> After you check I will perform RC5.
>>
>> All the code is on https://github.com/apache/incubator-wayang in the main
>> branch :D
>>
>> Best regards,
>> Bertty
>>
Re: PRE-RELEASE
Posted by jorge Arnulfo Quiané Ruiz <qu...@gmail.com>.
Hello guys,
I checked the list and everything looks good from my side. I am thus for
starting the RC5.
Thanks for the great job team!
Jorge
On Wed, Oct 27, 2021 at 2:58 PM Bertty Contreras <be...@scalytics.io>
wrote:
> Hello Folks,
>
> I did the modification on the code mentioned by Justin.
>
> Please double check the list and let me know if something is missing or
> wrong.
>
> After you check I will perform RC5.
>
> All the code is on https://github.com/apache/incubator-wayang in the main
> branch :D
>
> Best regards,
> Bertty
>
Re: PRE-RELEASE
Posted by Alexander Alten <al...@scalytics.io>.
Hi,
Rat found a file with unapproved licenses:
wayang/incubator-wayang/wayang-commons/wayang-utils-profile-db/NOTICE.md
The file should be NOTICE, and not NOTICE.md, imho. I created PR #35 to merge :) Hope that’ll fix it.
cheers,
—Alex
> On 27. Oct 2021, at 14:57, Bertty Contreras <be...@scalytics.io> wrote:
>
> Hello Folks,
>
> I did the modification on the code mentioned by Justin.
>
> Please double check the list and let me know if something is missing or
> wrong.
>
> After you check I will perform RC5.
>
> All the code is on https://github.com/apache/incubator-wayang in the main
> branch :D
>
> Best regards,
> Bertty