You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@madlib.apache.org by Satoshi Nagayasu <sn...@uptime.jp> on 2017/02/16 03:27:33 UTC
[VOTE] MADlib v1.10-rc1
Hello MADlib community,
We have created a MADlib 1.10 RC-1, with the artifacts below up for a vote.
This will be the 4th release for Apache MADlib (incubating).
The main goals of this release are:
* new modules (single source shortest path for graph analytics, encode
categorical variables, K-nearest neighbors)
* improvements to existing modules (add grouping support to elastic
net and PCA, add cross validation to elastic net, array input for
K-means, verbose output option for DT and RF, limit itemset size in
association rules, various madpack installer improvements)
* platform updates (PostgreSQL 9.6)
* bug fixes
* doc improvements
For more information including release notes, please see:
https://cwiki.apache.org/confluence/display/MADLIB/MADlib+1.10
*** Please download, review and vote by Sat Feb 18, 2017 @ 6pm PST ***
We're voting upon the source (tag): rc/1.10.0-rc1
https://github.com/apache/incubator-madlib/tree/rc/1.10.0-rc1
Source Files:
https://dist.apache.org/repos/dist/dev/incubator/madlib/1.10.0-incubating-rc1/
Commit to be voted upon:
https://github.com/apache/incubator-madlib/commit/ea17530bfe22a1fde173d7fa83508cbcd9924c20
KEYS file containing PGP Keys we use to sign the release:
https://dist.apache.org/repos/dist/dev/incubator/madlib/KEYS
To help in tallying the vote, can PMC members please be sure to
indicate "(binding)" with their vote.
[ ] +1 approve
[ ] +0 no opinion
[ ] -1 disapprove (and reason why)
--
Satoshi Nagayasu <sn...@uptime.jp>
Re: [VOTE] MADlib v1.10-rc1
Posted by Roman Shaposhnik <ro...@shaposhnik.org>.
On Wed, Feb 15, 2017 at 7:27 PM, Satoshi Nagayasu <sn...@uptime.jp> wrote:
> Hello MADlib community,
>
> We have created a MADlib 1.10 RC-1, with the artifacts below up for a vote.
>
> This will be the 4th release for Apache MADlib (incubating).
>
> The main goals of this release are:
> * new modules (single source shortest path for graph analytics, encode
> categorical variables, K-nearest neighbors)
> * improvements to existing modules (add grouping support to elastic
> net and PCA, add cross validation to elastic net, array input for
> K-means, verbose output option for DT and RF, limit itemset size in
> association rules, various madpack installer improvements)
> * platform updates (PostgreSQL 9.6)
> * bug fixes
> * doc improvements
>
> For more information including release notes, please see:
> https://cwiki.apache.org/confluence/display/MADLIB/MADlib+1.10
>
> *** Please download, review and vote by Sat Feb 18, 2017 @ 6pm PST ***
>
> We're voting upon the source (tag): rc/1.10.0-rc1
> https://github.com/apache/incubator-madlib/tree/rc/1.10.0-rc1
>
> Source Files:
> https://dist.apache.org/repos/dist/dev/incubator/madlib/1.10.0-incubating-rc1/
>
> Commit to be voted upon:
> https://github.com/apache/incubator-madlib/commit/ea17530bfe22a1fde173d7fa83508cbcd9924c20
>
> KEYS file containing PGP Keys we use to sign the release:
> https://dist.apache.org/repos/dist/dev/incubator/madlib/KEYS
>
> To help in tallying the vote, can PMC members please be sure to
> indicate "(binding)" with their vote.
>
> [ ] +1 approve
> [ ] +0 no opinion
> [ ] -1 disapprove (and reason why)
>
> --
> Satoshi Nagayasu <sn...@uptime.jp>
I'm afraid, I'll have to be -1
It seems that the following files were not a straight rename:
src/ports/postgres/modules/utilities/data_preparation.py_in ->
src/ports/postgres/modules/utilities/create_indicators.py_in
src/ports/postgres/modules/utilities/data_preparation.sql_in ->
src/ports/postgres/modules/utilities/create_indicators.sql_in
but in fact were modified quite a bit. As such we should really treat those
as new work that is now licensed under the ALv2 and requires ALv2
license header.
I know this dual situation with BSD/ALv2 is not ideal and I'm trying to
resolve it for us once and for all so we don't have to worry about things
like this one:
https://issues.apache.org/jira/browse/LEGAL-293
Since we'll have to do a re-spin, I'd like to also suggest that we take
care of the nits in Ed's feedback.
Hope this is OK with Satoshi.
Thanks,
Roman.
Re: [VOTE] MADlib v1.10-rc1
Posted by ChenLiang Wang <hi...@msn.com>.
+1
On 02/16/2017 11:27 AM, Satoshi Nagayasu wrote:
> Hello MADlib community,
>
> We have created a MADlib 1.10 RC-1, with the artifacts below up for a vote.
>
> This will be the 4th release for Apache MADlib (incubating).
>
> The main goals of this release are:
> * new modules (single source shortest path for graph analytics, encode
> categorical variables, K-nearest neighbors)
> * improvements to existing modules (add grouping support to elastic
> net and PCA, add cross validation to elastic net, array input for
> K-means, verbose output option for DT and RF, limit itemset size in
> association rules, various madpack installer improvements)
> * platform updates (PostgreSQL 9.6)
> * bug fixes
> * doc improvements
>
> For more information including release notes, please see:
> https://cwiki.apache.org/confluence/display/MADLIB/MADlib+1.10
>
> *** Please download, review and vote by Sat Feb 18, 2017 @ 6pm PST ***
>
> We're voting upon the source (tag): rc/1.10.0-rc1
> https://github.com/apache/incubator-madlib/tree/rc/1.10.0-rc1
>
> Source Files:
> https://dist.apache.org/repos/dist/dev/incubator/madlib/1.10.0-incubating-rc1/
>
> Commit to be voted upon:
> https://github.com/apache/incubator-madlib/commit/ea17530bfe22a1fde173d7fa83508cbcd9924c20
>
> KEYS file containing PGP Keys we use to sign the release:
> https://dist.apache.org/repos/dist/dev/incubator/madlib/KEYS
>
> To help in tallying the vote, can PMC members please be sure to
> indicate "(binding)" with their vote.
>
> [ ] +1 approve
> [ ] +0 no opinion
> [ ] -1 disapprove (and reason why)
>
Re: [VOTE] MADlib v1.10-rc1
Posted by Daisy Zhe Wang <da...@ufl.edu>.
+1
Daisy Zhe Wang
On Thu, 16 Feb 2017 09:17:56 -0800
Frank McQuillan <fm...@pivotal.io> wrote:
> +1
>
> Frank McQuillan
>
> On Wed, Feb 15, 2017 at 7:27 PM, Satoshi Nagayasu <sn...@uptime.jp>
> wrote:
>
> > Hello MADlib community,
> >
> > We have created a MADlib 1.10 RC-1, with the artifacts below up for
> > a vote.
> >
> > This will be the 4th release for Apache MADlib (incubating).
> >
> > The main goals of this release are:
> > * new modules (single source shortest path for graph analytics,
> > encode categorical variables, K-nearest neighbors)
> > * improvements to existing modules (add grouping support to elastic
> > net and PCA, add cross validation to elastic net, array input for
> > K-means, verbose output option for DT and RF, limit itemset size in
> > association rules, various madpack installer improvements)
> > * platform updates (PostgreSQL 9.6)
> > * bug fixes
> > * doc improvements
> >
> > For more information including release notes, please see:
> > https://cwiki.apache.org/confluence/display/MADLIB/MADlib+1.10
> >
> > *** Please download, review and vote by Sat Feb 18, 2017 @ 6pm PST
> > ***
> >
> > We're voting upon the source (tag): rc/1.10.0-rc1
> > https://github.com/apache/incubator-madlib/tree/rc/1.10.0-rc1
> >
> > Source Files:
> > https://dist.apache.org/repos/dist/dev/incubator/madlib/1.
> > 10.0-incubating-rc1/
> >
> > Commit to be voted upon:
> > https://github.com/apache/incubator-madlib/commit/
> > ea17530bfe22a1fde173d7fa83508cbcd9924c20
> >
> > KEYS file containing PGP Keys we use to sign the release:
> > https://dist.apache.org/repos/dist/dev/incubator/madlib/KEYS
> >
> > To help in tallying the vote, can PMC members please be sure to
> > indicate "(binding)" with their vote.
> >
> > [ ] +1 approve
> > [ ] +0 no opinion
> > [ ] -1 disapprove (and reason why)
> >
> > --
> > Satoshi Nagayasu <sn...@uptime.jp>
> >
>
Re: [VOTE] MADlib v1.10-rc1
Posted by "FENG, Xixuan (Aaron)" <xi...@gmail.com>.
+1
- Aaron
2017年2月17日(金) 3:05 Orhan Kislal <ok...@pivotal.io>:
> +1
>
> Orhan Kislal
>
> On Thu, Feb 16, 2017 at 9:23 AM, Joe Hellerstein <hellerstein@berkeley.edu
> > wrote:
>
> +1
>
> Sent from a telephone.
>
>
>
> > On Feb 16, 2017, at 9:17 AM, Frank McQuillan <fm...@pivotal.io>
> wrote:
> >
> > +1
> >
> > Frank McQuillan
> >
> >> On Wed, Feb 15, 2017 at 7:27 PM, Satoshi Nagayasu <sn...@uptime.jp>
> wrote:
> >>
> >> Hello MADlib community,
> >>
> >> We have created a MADlib 1.10 RC-1, with the artifacts below up for a
> vote.
> >>
> >> This will be the 4th release for Apache MADlib (incubating).
> >>
> >> The main goals of this release are:
> >> * new modules (single source shortest path for graph analytics, encode
> >> categorical variables, K-nearest neighbors)
> >> * improvements to existing modules (add grouping support to elastic
> >> net and PCA, add cross validation to elastic net, array input for
> >> K-means, verbose output option for DT and RF, limit itemset size in
> >> association rules, various madpack installer improvements)
> >> * platform updates (PostgreSQL 9.6)
> >> * bug fixes
> >> * doc improvements
> >>
> >> For more information including release notes, please see:
> >> https://cwiki.apache.org/confluence/display/MADLIB/MADlib+1.10
> >>
> >> *** Please download, review and vote by Sat Feb 18, 2017 @ 6pm PST ***
> >>
> >> We're voting upon the source (tag): rc/1.10.0-rc1
> >> https://github.com/apache/incubator-madlib/tree/rc/1.10.0-rc1
> >>
> >> Source Files:
> >> https://dist.apache.org/repos/dist/dev/incubator/madlib/1.
> >> 10.0-incubating-rc1/
> >>
> >> Commit to be voted upon:
> >> https://github.com/apache/incubator-madlib/commit/
> >> ea17530bfe22a1fde173d7fa83508cbcd9924c20
> >>
> >> KEYS file containing PGP Keys we use to sign the release:
> >> https://dist.apache.org/repos/dist/dev/incubator/madlib/KEYS
> >>
> >> To help in tallying the vote, can PMC members please be sure to
> >> indicate "(binding)" with their vote.
> >>
> >> [ ] +1 approve
> >> [ ] +0 no opinion
> >> [ ] -1 disapprove (and reason why)
> >>
> >> --
> >> Satoshi Nagayasu <sn...@uptime.jp>
> >>
>
>
Re: [VOTE] MADlib v1.10-rc1
Posted by "FENG, Xixuan (Aaron)" <xi...@gmail.com>.
+1
- Aaron
2017年2月17日(金) 3:05 Orhan Kislal <ok...@pivotal.io>:
> +1
>
> Orhan Kislal
>
> On Thu, Feb 16, 2017 at 9:23 AM, Joe Hellerstein <hellerstein@berkeley.edu
> > wrote:
>
> +1
>
> Sent from a telephone.
>
>
>
> > On Feb 16, 2017, at 9:17 AM, Frank McQuillan <fm...@pivotal.io>
> wrote:
> >
> > +1
> >
> > Frank McQuillan
> >
> >> On Wed, Feb 15, 2017 at 7:27 PM, Satoshi Nagayasu <sn...@uptime.jp>
> wrote:
> >>
> >> Hello MADlib community,
> >>
> >> We have created a MADlib 1.10 RC-1, with the artifacts below up for a
> vote.
> >>
> >> This will be the 4th release for Apache MADlib (incubating).
> >>
> >> The main goals of this release are:
> >> * new modules (single source shortest path for graph analytics, encode
> >> categorical variables, K-nearest neighbors)
> >> * improvements to existing modules (add grouping support to elastic
> >> net and PCA, add cross validation to elastic net, array input for
> >> K-means, verbose output option for DT and RF, limit itemset size in
> >> association rules, various madpack installer improvements)
> >> * platform updates (PostgreSQL 9.6)
> >> * bug fixes
> >> * doc improvements
> >>
> >> For more information including release notes, please see:
> >> https://cwiki.apache.org/confluence/display/MADLIB/MADlib+1.10
> >>
> >> *** Please download, review and vote by Sat Feb 18, 2017 @ 6pm PST ***
> >>
> >> We're voting upon the source (tag): rc/1.10.0-rc1
> >> https://github.com/apache/incubator-madlib/tree/rc/1.10.0-rc1
> >>
> >> Source Files:
> >> https://dist.apache.org/repos/dist/dev/incubator/madlib/1.
> >> 10.0-incubating-rc1/
> >>
> >> Commit to be voted upon:
> >> https://github.com/apache/incubator-madlib/commit/
> >> ea17530bfe22a1fde173d7fa83508cbcd9924c20
> >>
> >> KEYS file containing PGP Keys we use to sign the release:
> >> https://dist.apache.org/repos/dist/dev/incubator/madlib/KEYS
> >>
> >> To help in tallying the vote, can PMC members please be sure to
> >> indicate "(binding)" with their vote.
> >>
> >> [ ] +1 approve
> >> [ ] +0 no opinion
> >> [ ] -1 disapprove (and reason why)
> >>
> >> --
> >> Satoshi Nagayasu <sn...@uptime.jp>
> >>
>
>
Re: [VOTE] MADlib v1.10-rc1
Posted by Nandish Jayaram <nj...@pivotal.io>.
+1
NJ
Sent from my iPhone
> On Feb 16, 2017, at 12:06 PM, Orhan Kislal <ok...@pivotal.io> wrote:
>
> Hi Ed,
>
> Thanks for the review. One of the comments from the previous release was a preference towards a signature with an Apache id. Since Satoshi-san is not an Apache committer yet, I took care of the signing process.
>
> Thanks,
>
> Orhan Kislal
>
>> On Thu, Feb 16, 2017 at 11:58 AM, Ed Espino <es...@apache.org> wrote:
>> A few MADlib v1.10-rc1 observations from a HAWQ incubator committer.
>>
>> - The Copyright year (2016) in the NOTICE file needs to be updated to
>> 2017. I believe this can be handled in next release.
>> - As it still applies, similar to a past comment by Roman ([VOTE] MADlib
>> v1.9.1-rc2
>> <https://lists.apache.org/thread.html/981b4c24eaa2ab069b8e18f7aa4bddc7a78d3a9dc26bf659af94fcfe@%3Cgeneral.incubator.apache.org%3E>)
>> - *"* name of the top level folder in the archive is weird. The usual
>> practice is to call the top level folder as <project name>-<version ID>*"*
>> (example: *apache-madlib-src-1.10-incubating* instead of
>> *incubator-madlib*)
>> - I'm more curious than anything. Why did Orhan sign the release? I was
>> expecting the release manager (Satoshi Nagayasu) to have signed the release.
>> - Checksums and PGP signature are good.
>> - ASF headers check: I spot checked files added (git whatchanged
>> --diff-filter=A) since the last release. ASF headers look good. Nice Job!
>>
>> I was going to try and build but I ran past my allotted time limit for this
>> review. Hopefully, I can try this soon.
>>
>> Regards,
>> -=ed espino
>>
>> On Thu, Feb 16, 2017 at 10:05 AM, Orhan Kislal <ok...@pivotal.io> wrote:
>>
>> > +1
>> >
>> > Orhan Kislal
>> >
>> > On Thu, Feb 16, 2017 at 9:23 AM, Joe Hellerstein <hellerstein@berkeley.edu
>> > >
>> > wrote:
>> >
>> > > +1
>> > >
>> > > Sent from a telephone.
>> > >
>> > > > On Feb 16, 2017, at 9:17 AM, Frank McQuillan <fm...@pivotal.io>
>> > > wrote:
>> > > >
>> > > > +1
>> > > >
>> > > > Frank McQuillan
>> > > >
>> > > >> On Wed, Feb 15, 2017 at 7:27 PM, Satoshi Nagayasu <sn...@uptime.jp>
>> > > wrote:
>> > > >>
>> > > >> Hello MADlib community,
>> > > >>
>> > > >> We have created a MADlib 1.10 RC-1, with the artifacts below up for a
>> > > vote.
>> > > >>
>> > > >> This will be the 4th release for Apache MADlib (incubating).
>> > > >>
>> > > >> The main goals of this release are:
>> > > >> * new modules (single source shortest path for graph analytics, encode
>> > > >> categorical variables, K-nearest neighbors)
>> > > >> * improvements to existing modules (add grouping support to elastic
>> > > >> net and PCA, add cross validation to elastic net, array input for
>> > > >> K-means, verbose output option for DT and RF, limit itemset size in
>> > > >> association rules, various madpack installer improvements)
>> > > >> * platform updates (PostgreSQL 9.6)
>> > > >> * bug fixes
>> > > >> * doc improvements
>> > > >>
>> > > >> For more information including release notes, please see:
>> > > >> https://cwiki.apache.org/confluence/display/MADLIB/MADlib+1.10
>> > > >>
>> > > >> *** Please download, review and vote by Sat Feb 18, 2017 @ 6pm PST ***
>> > > >>
>> > > >> We're voting upon the source (tag): rc/1.10.0-rc1
>> > > >> https://github.com/apache/incubator-madlib/tree/rc/1.10.0-rc1
>> > > >>
>> > > >> Source Files:
>> > > >> https://dist.apache.org/repos/dist/dev/incubator/madlib/1.
>> > > >> 10.0-incubating-rc1/
>> > > >>
>> > > >> Commit to be voted upon:
>> > > >> https://github.com/apache/incubator-madlib/commit/
>> > > >> ea17530bfe22a1fde173d7fa83508cbcd9924c20
>> > > >>
>> > > >> KEYS file containing PGP Keys we use to sign the release:
>> > > >> https://dist.apache.org/repos/dist/dev/incubator/madlib/KEYS
>> > > >>
>> > > >> To help in tallying the vote, can PMC members please be sure to
>> > > >> indicate "(binding)" with their vote.
>> > > >>
>> > > >> [ ] +1 approve
>> > > >> [ ] +0 no opinion
>> > > >> [ ] -1 disapprove (and reason why)
>> > > >>
>> > > >> --
>> > > >> Satoshi Nagayasu <sn...@uptime.jp>
>> > > >>
>> > >
>> >
>>
>>
>>
>> --
>> *Ed Espino*
>> *espino@apache.org <es...@apache.org>*
>
Re: [VOTE] MADlib v1.10-rc1
Posted by Frank McQuillan <fm...@pivotal.io>.
To finish this thread, I captured all of these licensing issues on the
MADlib wiki at
https://cwiki.apache.org/confluence/display/MADLIB/ASF+Licensing+Guidance
should anyone need to refer to it.
On Tue, Feb 28, 2017 at 11:43 AM, Frank McQuillan <fm...@pivotal.io>
wrote:
> Thanks Rahul. I see your commit has addressed the remaining issues:
> https://git1-us-west.apache.org/repos/asf?p=incubator-
> madlib.git;a=commit;h=a3863b6c
>
> We are declaring create_indicators.* as new files so they will have
> Apache header.
>
> For the record, I attached an Excel spreadsheet with some more notes so
> that we remember how we went from the two lists Rahul posted above to the
> above commit.
>
> Frank
>
> On Mon, Feb 27, 2017 at 5:44 PM, Rahul Iyer <ra...@gmail.com> wrote:
>
>> I have attached two files:
>>
>> new_files_after_apache.txt: New files added since September 15, 2015
>> (grant date) till date
>> files_w_apache_header.txt: Files that contain the Apache header right
>> now.
>>
>> Comparing the two lists, there are open questions regarding below files.
>>
>> Extra headers:
>> - sort-module.py has Apache header but was created before grant (recently
>> edited and header added). *I'll fix this*.
>> - create_indicators.* have headers but were renamed from
>> data_preparation.*. *What is the legal guidance with this*?
>>
>> No header:
>> - class_diagram.mp looks like a text file with no header, even though it
>> was added just after the grant. I'm not aware of the purpose of this file.
>>
>>
>>
>> On Mon, Feb 27, 2017 at 4:42 PM, Frank McQuillan <fm...@pivotal.io>
>> wrote:
>>
>>> OK, so we need to go back and do the comparison from the original code
>>> grant in the fall of 2015 to the current 1.10 release candidate.
>>>
>>> On Mon, Feb 27, 2017 at 4:19 PM, Roman Shaposhnik <ro...@shaposhnik.org>
>>> wrote:
>>>
>>> > Frank, I'm not sure I understand the question. The criteria needs to
>>> hold
>>> > for anything that came in via the initial code ingest compared to how
>>> the
>>> > master of your project looks now.
>>> >
>>> > Thanks,
>>> > Roman.
>>> >
>>> > On Mon, Feb 27, 2017 at 4:10 PM, Frank McQuillan <
>>> fmcquillan@pivotal.io>
>>> > wrote:
>>> > > Roman,
>>> > >
>>> > > Does this apply retro-actively back to initial grant of the code to
>>> > ASF? Or
>>> > > just from the last release 1.9.1?
>>> > >
>>> > > Frank
>>> > >
>>> > > On Sun, Feb 26, 2017 at 11:23 PM, Roman Shaposhnik <
>>> roman@shaposhnik.org
>>> > >
>>> > > wrote:
>>> > >>
>>> > >> Here's the ultimate resolution on the IP issue:
>>> > >> * we don't do anything with existing (BSD) files even if we edit
>>> them
>>> > >> * every new file we create gets an ASF license header
>>> > >>
>>> > >> More details:
>>> > >>
>>> > >> https://issues.apache.org/jira/browse/LEGAL-293?
>>> > focusedCommentId=15881595&page=com.atlassian.jira.
>>> > plugin.system.issuetabpanels:comment-tabpanel#comment-15881595
>>> > >>
>>> > >> Thanks,
>>> > >> Roman.
>>> > >>
>>> > >> On Tue, Feb 21, 2017 at 5:54 PM, Frank McQuillan <
>>> fmcquillan@pivotal.io
>>> > >
>>> > >> wrote:
>>> > >> > Thanks Roman for working on this.
>>> > >> >
>>> > >> > If you feel a final answer will be ready next week, then yes by
>>> all
>>> > >> > means l
>>> > >> > would suggest to the community that we wait and re-spin an RC2
>>> with
>>> > the
>>> > >> > license headers issue resolved. Seems less overhead and effort
>>> than a
>>> > >> > quick follow on release right after 1.10. Also, there some
>>> momentum
>>> > >> > going
>>> > >> > with the legal discussion, so let's take advantage of that.
>>> > >> >
>>> > >> > Satoshi (release manager), are you OK pausing the RC2 until we
>>> hear
>>> > back
>>> > >> > from Roman next week?
>>> > >> >
>>> > >> > Thank you,
>>> > >> > Frank
>>> > >> >
>>> > >> >
>>> > >> > On Tue, Feb 21, 2017 at 4:45 PM, Roman Shaposhnik <
>>> > roman@shaposhnik.org>
>>> > >> > wrote:
>>> > >> >
>>> > >> >> On Tue, Feb 21, 2017 at 2:55 PM, Frank McQuillan
>>> > >> >> <fm...@pivotal.io>
>>> > >> >> wrote:
>>> > >> >> > Agree with Rahul re putting up an RC2 with the suggested
>>> changes
>>> > from
>>> > >> >> Roman,
>>> > >> >> > including incorporating Ed's comments on copyright year and top
>>> > level
>>> > >> >> folder
>>> > >> >> > naming. These are really items but let's respond to the RC1
>>> > >> >> > reviewers
>>> > >> >> the
>>> > >> >> > best way we can.
>>> > >> >>
>>> > >> >> +1 to a respin.
>>> > >> >>
>>> > >> >> > Regarding the ASF legal issue being discussed, MADLib
>>> community is
>>> > >> >> > more
>>> > >> >> than
>>> > >> >> > happy to respond to any guidance from the fine folks at the ASF
>>> > >> >> > around
>>> > >> >> > headers with appropriate licensing verbage. We just need to
>>> know
>>> > >> >> > what
>>> > >> >> that
>>> > >> >> > guidance is.
>>> > >> >>
>>> > >> >> Well, if you're ok respinning next week I hope to get you a final
>>> > >> >> answer by then.
>>> > >> >> Might as well kill two birds with the same RC. Or we can quickly
>>> do a
>>> > >> >> follow up
>>> > >> >> release once the licensing headers dust settles. Up to you guys.
>>> > >> >>
>>> > >> >> Thanks,
>>> > >> >> Roman.
>>> > >> >>
>>> > >
>>> > >
>>> >
>>>
>>
>>
>
Re: [VOTE] MADlib v1.10-rc1
Posted by Frank McQuillan <fm...@pivotal.io>.
To finish this thread, I captured all of these licensing issues on the
MADlib wiki at
https://cwiki.apache.org/confluence/display/MADLIB/ASF+Licensing+Guidance
should anyone need to refer to it.
On Tue, Feb 28, 2017 at 11:43 AM, Frank McQuillan <fm...@pivotal.io>
wrote:
> Thanks Rahul. I see your commit has addressed the remaining issues:
> https://git1-us-west.apache.org/repos/asf?p=incubator-
> madlib.git;a=commit;h=a3863b6c
>
> We are declaring create_indicators.* as new files so they will have
> Apache header.
>
> For the record, I attached an Excel spreadsheet with some more notes so
> that we remember how we went from the two lists Rahul posted above to the
> above commit.
>
> Frank
>
> On Mon, Feb 27, 2017 at 5:44 PM, Rahul Iyer <ra...@gmail.com> wrote:
>
>> I have attached two files:
>>
>> new_files_after_apache.txt: New files added since September 15, 2015
>> (grant date) till date
>> files_w_apache_header.txt: Files that contain the Apache header right
>> now.
>>
>> Comparing the two lists, there are open questions regarding below files.
>>
>> Extra headers:
>> - sort-module.py has Apache header but was created before grant (recently
>> edited and header added). *I'll fix this*.
>> - create_indicators.* have headers but were renamed from
>> data_preparation.*. *What is the legal guidance with this*?
>>
>> No header:
>> - class_diagram.mp looks like a text file with no header, even though it
>> was added just after the grant. I'm not aware of the purpose of this file.
>>
>>
>>
>> On Mon, Feb 27, 2017 at 4:42 PM, Frank McQuillan <fm...@pivotal.io>
>> wrote:
>>
>>> OK, so we need to go back and do the comparison from the original code
>>> grant in the fall of 2015 to the current 1.10 release candidate.
>>>
>>> On Mon, Feb 27, 2017 at 4:19 PM, Roman Shaposhnik <ro...@shaposhnik.org>
>>> wrote:
>>>
>>> > Frank, I'm not sure I understand the question. The criteria needs to
>>> hold
>>> > for anything that came in via the initial code ingest compared to how
>>> the
>>> > master of your project looks now.
>>> >
>>> > Thanks,
>>> > Roman.
>>> >
>>> > On Mon, Feb 27, 2017 at 4:10 PM, Frank McQuillan <
>>> fmcquillan@pivotal.io>
>>> > wrote:
>>> > > Roman,
>>> > >
>>> > > Does this apply retro-actively back to initial grant of the code to
>>> > ASF? Or
>>> > > just from the last release 1.9.1?
>>> > >
>>> > > Frank
>>> > >
>>> > > On Sun, Feb 26, 2017 at 11:23 PM, Roman Shaposhnik <
>>> roman@shaposhnik.org
>>> > >
>>> > > wrote:
>>> > >>
>>> > >> Here's the ultimate resolution on the IP issue:
>>> > >> * we don't do anything with existing (BSD) files even if we edit
>>> them
>>> > >> * every new file we create gets an ASF license header
>>> > >>
>>> > >> More details:
>>> > >>
>>> > >> https://issues.apache.org/jira/browse/LEGAL-293?
>>> > focusedCommentId=15881595&page=com.atlassian.jira.
>>> > plugin.system.issuetabpanels:comment-tabpanel#comment-15881595
>>> > >>
>>> > >> Thanks,
>>> > >> Roman.
>>> > >>
>>> > >> On Tue, Feb 21, 2017 at 5:54 PM, Frank McQuillan <
>>> fmcquillan@pivotal.io
>>> > >
>>> > >> wrote:
>>> > >> > Thanks Roman for working on this.
>>> > >> >
>>> > >> > If you feel a final answer will be ready next week, then yes by
>>> all
>>> > >> > means l
>>> > >> > would suggest to the community that we wait and re-spin an RC2
>>> with
>>> > the
>>> > >> > license headers issue resolved. Seems less overhead and effort
>>> than a
>>> > >> > quick follow on release right after 1.10. Also, there some
>>> momentum
>>> > >> > going
>>> > >> > with the legal discussion, so let's take advantage of that.
>>> > >> >
>>> > >> > Satoshi (release manager), are you OK pausing the RC2 until we
>>> hear
>>> > back
>>> > >> > from Roman next week?
>>> > >> >
>>> > >> > Thank you,
>>> > >> > Frank
>>> > >> >
>>> > >> >
>>> > >> > On Tue, Feb 21, 2017 at 4:45 PM, Roman Shaposhnik <
>>> > roman@shaposhnik.org>
>>> > >> > wrote:
>>> > >> >
>>> > >> >> On Tue, Feb 21, 2017 at 2:55 PM, Frank McQuillan
>>> > >> >> <fm...@pivotal.io>
>>> > >> >> wrote:
>>> > >> >> > Agree with Rahul re putting up an RC2 with the suggested
>>> changes
>>> > from
>>> > >> >> Roman,
>>> > >> >> > including incorporating Ed's comments on copyright year and top
>>> > level
>>> > >> >> folder
>>> > >> >> > naming. These are really items but let's respond to the RC1
>>> > >> >> > reviewers
>>> > >> >> the
>>> > >> >> > best way we can.
>>> > >> >>
>>> > >> >> +1 to a respin.
>>> > >> >>
>>> > >> >> > Regarding the ASF legal issue being discussed, MADLib
>>> community is
>>> > >> >> > more
>>> > >> >> than
>>> > >> >> > happy to respond to any guidance from the fine folks at the ASF
>>> > >> >> > around
>>> > >> >> > headers with appropriate licensing verbage. We just need to
>>> know
>>> > >> >> > what
>>> > >> >> that
>>> > >> >> > guidance is.
>>> > >> >>
>>> > >> >> Well, if you're ok respinning next week I hope to get you a final
>>> > >> >> answer by then.
>>> > >> >> Might as well kill two birds with the same RC. Or we can quickly
>>> do a
>>> > >> >> follow up
>>> > >> >> release once the licensing headers dust settles. Up to you guys.
>>> > >> >>
>>> > >> >> Thanks,
>>> > >> >> Roman.
>>> > >> >>
>>> > >
>>> > >
>>> >
>>>
>>
>>
>
Re: [VOTE] MADlib v1.10-rc1
Posted by Frank McQuillan <fm...@pivotal.io>.
Thanks Rahul. I see your commit has addressed the remaining issues:
https://git1-us-west.apache.org/repos/asf?p=incubator-madlib.git;a=commit;h=a3863b6c
We are declaring create_indicators.* as new files so they will have Apache
header.
For the record, I attached an Excel spreadsheet with some more notes so
that we remember how we went from the two lists Rahul posted above to the
above commit.
Frank
On Mon, Feb 27, 2017 at 5:44 PM, Rahul Iyer <ra...@gmail.com> wrote:
> I have attached two files:
>
> new_files_after_apache.txt: New files added since September 15, 2015
> (grant date) till date
> files_w_apache_header.txt: Files that contain the Apache header right now.
>
> Comparing the two lists, there are open questions regarding below files.
>
> Extra headers:
> - sort-module.py has Apache header but was created before grant (recently
> edited and header added). *I'll fix this*.
> - create_indicators.* have headers but were renamed from
> data_preparation.*. *What is the legal guidance with this*?
>
> No header:
> - class_diagram.mp looks like a text file with no header, even though it
> was added just after the grant. I'm not aware of the purpose of this file.
>
>
>
> On Mon, Feb 27, 2017 at 4:42 PM, Frank McQuillan <fm...@pivotal.io>
> wrote:
>
>> OK, so we need to go back and do the comparison from the original code
>> grant in the fall of 2015 to the current 1.10 release candidate.
>>
>> On Mon, Feb 27, 2017 at 4:19 PM, Roman Shaposhnik <ro...@shaposhnik.org>
>> wrote:
>>
>> > Frank, I'm not sure I understand the question. The criteria needs to
>> hold
>> > for anything that came in via the initial code ingest compared to how
>> the
>> > master of your project looks now.
>> >
>> > Thanks,
>> > Roman.
>> >
>> > On Mon, Feb 27, 2017 at 4:10 PM, Frank McQuillan <fmcquillan@pivotal.io
>> >
>> > wrote:
>> > > Roman,
>> > >
>> > > Does this apply retro-actively back to initial grant of the code to
>> > ASF? Or
>> > > just from the last release 1.9.1?
>> > >
>> > > Frank
>> > >
>> > > On Sun, Feb 26, 2017 at 11:23 PM, Roman Shaposhnik <
>> roman@shaposhnik.org
>> > >
>> > > wrote:
>> > >>
>> > >> Here's the ultimate resolution on the IP issue:
>> > >> * we don't do anything with existing (BSD) files even if we edit
>> them
>> > >> * every new file we create gets an ASF license header
>> > >>
>> > >> More details:
>> > >>
>> > >> https://issues.apache.org/jira/browse/LEGAL-293?
>> > focusedCommentId=15881595&page=com.atlassian.jira.
>> > plugin.system.issuetabpanels:comment-tabpanel#comment-15881595
>> > >>
>> > >> Thanks,
>> > >> Roman.
>> > >>
>> > >> On Tue, Feb 21, 2017 at 5:54 PM, Frank McQuillan <
>> fmcquillan@pivotal.io
>> > >
>> > >> wrote:
>> > >> > Thanks Roman for working on this.
>> > >> >
>> > >> > If you feel a final answer will be ready next week, then yes by all
>> > >> > means l
>> > >> > would suggest to the community that we wait and re-spin an RC2 with
>> > the
>> > >> > license headers issue resolved. Seems less overhead and effort
>> than a
>> > >> > quick follow on release right after 1.10. Also, there some
>> momentum
>> > >> > going
>> > >> > with the legal discussion, so let's take advantage of that.
>> > >> >
>> > >> > Satoshi (release manager), are you OK pausing the RC2 until we hear
>> > back
>> > >> > from Roman next week?
>> > >> >
>> > >> > Thank you,
>> > >> > Frank
>> > >> >
>> > >> >
>> > >> > On Tue, Feb 21, 2017 at 4:45 PM, Roman Shaposhnik <
>> > roman@shaposhnik.org>
>> > >> > wrote:
>> > >> >
>> > >> >> On Tue, Feb 21, 2017 at 2:55 PM, Frank McQuillan
>> > >> >> <fm...@pivotal.io>
>> > >> >> wrote:
>> > >> >> > Agree with Rahul re putting up an RC2 with the suggested changes
>> > from
>> > >> >> Roman,
>> > >> >> > including incorporating Ed's comments on copyright year and top
>> > level
>> > >> >> folder
>> > >> >> > naming. These are really items but let's respond to the RC1
>> > >> >> > reviewers
>> > >> >> the
>> > >> >> > best way we can.
>> > >> >>
>> > >> >> +1 to a respin.
>> > >> >>
>> > >> >> > Regarding the ASF legal issue being discussed, MADLib community
>> is
>> > >> >> > more
>> > >> >> than
>> > >> >> > happy to respond to any guidance from the fine folks at the ASF
>> > >> >> > around
>> > >> >> > headers with appropriate licensing verbage. We just need to
>> know
>> > >> >> > what
>> > >> >> that
>> > >> >> > guidance is.
>> > >> >>
>> > >> >> Well, if you're ok respinning next week I hope to get you a final
>> > >> >> answer by then.
>> > >> >> Might as well kill two birds with the same RC. Or we can quickly
>> do a
>> > >> >> follow up
>> > >> >> release once the licensing headers dust settles. Up to you guys.
>> > >> >>
>> > >> >> Thanks,
>> > >> >> Roman.
>> > >> >>
>> > >
>> > >
>> >
>>
>
>
Re: [VOTE] MADlib v1.10-rc1
Posted by Frank McQuillan <fm...@pivotal.io>.
Thanks Rahul. I see your commit has addressed the remaining issues:
https://git1-us-west.apache.org/repos/asf?p=incubator-madlib.git;a=commit;h=a3863b6c
We are declaring create_indicators.* as new files so they will have Apache
header.
For the record, I attached an Excel spreadsheet with some more notes so
that we remember how we went from the two lists Rahul posted above to the
above commit.
Frank
On Mon, Feb 27, 2017 at 5:44 PM, Rahul Iyer <ra...@gmail.com> wrote:
> I have attached two files:
>
> new_files_after_apache.txt: New files added since September 15, 2015
> (grant date) till date
> files_w_apache_header.txt: Files that contain the Apache header right now.
>
> Comparing the two lists, there are open questions regarding below files.
>
> Extra headers:
> - sort-module.py has Apache header but was created before grant (recently
> edited and header added). *I'll fix this*.
> - create_indicators.* have headers but were renamed from
> data_preparation.*. *What is the legal guidance with this*?
>
> No header:
> - class_diagram.mp looks like a text file with no header, even though it
> was added just after the grant. I'm not aware of the purpose of this file.
>
>
>
> On Mon, Feb 27, 2017 at 4:42 PM, Frank McQuillan <fm...@pivotal.io>
> wrote:
>
>> OK, so we need to go back and do the comparison from the original code
>> grant in the fall of 2015 to the current 1.10 release candidate.
>>
>> On Mon, Feb 27, 2017 at 4:19 PM, Roman Shaposhnik <ro...@shaposhnik.org>
>> wrote:
>>
>> > Frank, I'm not sure I understand the question. The criteria needs to
>> hold
>> > for anything that came in via the initial code ingest compared to how
>> the
>> > master of your project looks now.
>> >
>> > Thanks,
>> > Roman.
>> >
>> > On Mon, Feb 27, 2017 at 4:10 PM, Frank McQuillan <fmcquillan@pivotal.io
>> >
>> > wrote:
>> > > Roman,
>> > >
>> > > Does this apply retro-actively back to initial grant of the code to
>> > ASF? Or
>> > > just from the last release 1.9.1?
>> > >
>> > > Frank
>> > >
>> > > On Sun, Feb 26, 2017 at 11:23 PM, Roman Shaposhnik <
>> roman@shaposhnik.org
>> > >
>> > > wrote:
>> > >>
>> > >> Here's the ultimate resolution on the IP issue:
>> > >> * we don't do anything with existing (BSD) files even if we edit
>> them
>> > >> * every new file we create gets an ASF license header
>> > >>
>> > >> More details:
>> > >>
>> > >> https://issues.apache.org/jira/browse/LEGAL-293?
>> > focusedCommentId=15881595&page=com.atlassian.jira.
>> > plugin.system.issuetabpanels:comment-tabpanel#comment-15881595
>> > >>
>> > >> Thanks,
>> > >> Roman.
>> > >>
>> > >> On Tue, Feb 21, 2017 at 5:54 PM, Frank McQuillan <
>> fmcquillan@pivotal.io
>> > >
>> > >> wrote:
>> > >> > Thanks Roman for working on this.
>> > >> >
>> > >> > If you feel a final answer will be ready next week, then yes by all
>> > >> > means l
>> > >> > would suggest to the community that we wait and re-spin an RC2 with
>> > the
>> > >> > license headers issue resolved. Seems less overhead and effort
>> than a
>> > >> > quick follow on release right after 1.10. Also, there some
>> momentum
>> > >> > going
>> > >> > with the legal discussion, so let's take advantage of that.
>> > >> >
>> > >> > Satoshi (release manager), are you OK pausing the RC2 until we hear
>> > back
>> > >> > from Roman next week?
>> > >> >
>> > >> > Thank you,
>> > >> > Frank
>> > >> >
>> > >> >
>> > >> > On Tue, Feb 21, 2017 at 4:45 PM, Roman Shaposhnik <
>> > roman@shaposhnik.org>
>> > >> > wrote:
>> > >> >
>> > >> >> On Tue, Feb 21, 2017 at 2:55 PM, Frank McQuillan
>> > >> >> <fm...@pivotal.io>
>> > >> >> wrote:
>> > >> >> > Agree with Rahul re putting up an RC2 with the suggested changes
>> > from
>> > >> >> Roman,
>> > >> >> > including incorporating Ed's comments on copyright year and top
>> > level
>> > >> >> folder
>> > >> >> > naming. These are really items but let's respond to the RC1
>> > >> >> > reviewers
>> > >> >> the
>> > >> >> > best way we can.
>> > >> >>
>> > >> >> +1 to a respin.
>> > >> >>
>> > >> >> > Regarding the ASF legal issue being discussed, MADLib community
>> is
>> > >> >> > more
>> > >> >> than
>> > >> >> > happy to respond to any guidance from the fine folks at the ASF
>> > >> >> > around
>> > >> >> > headers with appropriate licensing verbage. We just need to
>> know
>> > >> >> > what
>> > >> >> that
>> > >> >> > guidance is.
>> > >> >>
>> > >> >> Well, if you're ok respinning next week I hope to get you a final
>> > >> >> answer by then.
>> > >> >> Might as well kill two birds with the same RC. Or we can quickly
>> do a
>> > >> >> follow up
>> > >> >> release once the licensing headers dust settles. Up to you guys.
>> > >> >>
>> > >> >> Thanks,
>> > >> >> Roman.
>> > >> >>
>> > >
>> > >
>> >
>>
>
>
Re: [VOTE] MADlib v1.10-rc1
Posted by Rahul Iyer <ra...@gmail.com>.
I have attached two files:
new_files_after_apache.txt: New files added since September 15, 2015 (grant
date) till date
files_w_apache_header.txt: Files that contain the Apache header right now.
Comparing the two lists, there are open questions regarding below files.
Extra headers:
- sort-module.py has Apache header but was created before grant (recently
edited and header added). *I'll fix this*.
- create_indicators.* have headers but were renamed from
data_preparation.*. *What is the legal guidance with this*?
No header:
- class_diagram.mp looks like a text file with no header, even though it
was added just after the grant. I'm not aware of the purpose of this file.
On Mon, Feb 27, 2017 at 4:42 PM, Frank McQuillan <fm...@pivotal.io>
wrote:
> OK, so we need to go back and do the comparison from the original code
> grant in the fall of 2015 to the current 1.10 release candidate.
>
> On Mon, Feb 27, 2017 at 4:19 PM, Roman Shaposhnik <ro...@shaposhnik.org>
> wrote:
>
> > Frank, I'm not sure I understand the question. The criteria needs to hold
> > for anything that came in via the initial code ingest compared to how the
> > master of your project looks now.
> >
> > Thanks,
> > Roman.
> >
> > On Mon, Feb 27, 2017 at 4:10 PM, Frank McQuillan <fm...@pivotal.io>
> > wrote:
> > > Roman,
> > >
> > > Does this apply retro-actively back to initial grant of the code to
> > ASF? Or
> > > just from the last release 1.9.1?
> > >
> > > Frank
> > >
> > > On Sun, Feb 26, 2017 at 11:23 PM, Roman Shaposhnik <
> roman@shaposhnik.org
> > >
> > > wrote:
> > >>
> > >> Here's the ultimate resolution on the IP issue:
> > >> * we don't do anything with existing (BSD) files even if we edit
> them
> > >> * every new file we create gets an ASF license header
> > >>
> > >> More details:
> > >>
> > >> https://issues.apache.org/jira/browse/LEGAL-293?
> > focusedCommentId=15881595&page=com.atlassian.jira.
> > plugin.system.issuetabpanels:comment-tabpanel#comment-15881595
> > >>
> > >> Thanks,
> > >> Roman.
> > >>
> > >> On Tue, Feb 21, 2017 at 5:54 PM, Frank McQuillan <
> fmcquillan@pivotal.io
> > >
> > >> wrote:
> > >> > Thanks Roman for working on this.
> > >> >
> > >> > If you feel a final answer will be ready next week, then yes by all
> > >> > means l
> > >> > would suggest to the community that we wait and re-spin an RC2 with
> > the
> > >> > license headers issue resolved. Seems less overhead and effort
> than a
> > >> > quick follow on release right after 1.10. Also, there some momentum
> > >> > going
> > >> > with the legal discussion, so let's take advantage of that.
> > >> >
> > >> > Satoshi (release manager), are you OK pausing the RC2 until we hear
> > back
> > >> > from Roman next week?
> > >> >
> > >> > Thank you,
> > >> > Frank
> > >> >
> > >> >
> > >> > On Tue, Feb 21, 2017 at 4:45 PM, Roman Shaposhnik <
> > roman@shaposhnik.org>
> > >> > wrote:
> > >> >
> > >> >> On Tue, Feb 21, 2017 at 2:55 PM, Frank McQuillan
> > >> >> <fm...@pivotal.io>
> > >> >> wrote:
> > >> >> > Agree with Rahul re putting up an RC2 with the suggested changes
> > from
> > >> >> Roman,
> > >> >> > including incorporating Ed's comments on copyright year and top
> > level
> > >> >> folder
> > >> >> > naming. These are really items but let's respond to the RC1
> > >> >> > reviewers
> > >> >> the
> > >> >> > best way we can.
> > >> >>
> > >> >> +1 to a respin.
> > >> >>
> > >> >> > Regarding the ASF legal issue being discussed, MADLib community
> is
> > >> >> > more
> > >> >> than
> > >> >> > happy to respond to any guidance from the fine folks at the ASF
> > >> >> > around
> > >> >> > headers with appropriate licensing verbage. We just need to know
> > >> >> > what
> > >> >> that
> > >> >> > guidance is.
> > >> >>
> > >> >> Well, if you're ok respinning next week I hope to get you a final
> > >> >> answer by then.
> > >> >> Might as well kill two birds with the same RC. Or we can quickly
> do a
> > >> >> follow up
> > >> >> release once the licensing headers dust settles. Up to you guys.
> > >> >>
> > >> >> Thanks,
> > >> >> Roman.
> > >> >>
> > >
> > >
> >
>
Re: [VOTE] MADlib v1.10-rc1
Posted by Rahul Iyer <ra...@gmail.com>.
I have attached two files:
new_files_after_apache.txt: New files added since September 15, 2015 (grant
date) till date
files_w_apache_header.txt: Files that contain the Apache header right now.
Comparing the two lists, there are open questions regarding below files.
Extra headers:
- sort-module.py has Apache header but was created before grant (recently
edited and header added). *I'll fix this*.
- create_indicators.* have headers but were renamed from
data_preparation.*. *What is the legal guidance with this*?
No header:
- class_diagram.mp looks like a text file with no header, even though it
was added just after the grant. I'm not aware of the purpose of this file.
On Mon, Feb 27, 2017 at 4:42 PM, Frank McQuillan <fm...@pivotal.io>
wrote:
> OK, so we need to go back and do the comparison from the original code
> grant in the fall of 2015 to the current 1.10 release candidate.
>
> On Mon, Feb 27, 2017 at 4:19 PM, Roman Shaposhnik <ro...@shaposhnik.org>
> wrote:
>
> > Frank, I'm not sure I understand the question. The criteria needs to hold
> > for anything that came in via the initial code ingest compared to how the
> > master of your project looks now.
> >
> > Thanks,
> > Roman.
> >
> > On Mon, Feb 27, 2017 at 4:10 PM, Frank McQuillan <fm...@pivotal.io>
> > wrote:
> > > Roman,
> > >
> > > Does this apply retro-actively back to initial grant of the code to
> > ASF? Or
> > > just from the last release 1.9.1?
> > >
> > > Frank
> > >
> > > On Sun, Feb 26, 2017 at 11:23 PM, Roman Shaposhnik <
> roman@shaposhnik.org
> > >
> > > wrote:
> > >>
> > >> Here's the ultimate resolution on the IP issue:
> > >> * we don't do anything with existing (BSD) files even if we edit
> them
> > >> * every new file we create gets an ASF license header
> > >>
> > >> More details:
> > >>
> > >> https://issues.apache.org/jira/browse/LEGAL-293?
> > focusedCommentId=15881595&page=com.atlassian.jira.
> > plugin.system.issuetabpanels:comment-tabpanel#comment-15881595
> > >>
> > >> Thanks,
> > >> Roman.
> > >>
> > >> On Tue, Feb 21, 2017 at 5:54 PM, Frank McQuillan <
> fmcquillan@pivotal.io
> > >
> > >> wrote:
> > >> > Thanks Roman for working on this.
> > >> >
> > >> > If you feel a final answer will be ready next week, then yes by all
> > >> > means l
> > >> > would suggest to the community that we wait and re-spin an RC2 with
> > the
> > >> > license headers issue resolved. Seems less overhead and effort
> than a
> > >> > quick follow on release right after 1.10. Also, there some momentum
> > >> > going
> > >> > with the legal discussion, so let's take advantage of that.
> > >> >
> > >> > Satoshi (release manager), are you OK pausing the RC2 until we hear
> > back
> > >> > from Roman next week?
> > >> >
> > >> > Thank you,
> > >> > Frank
> > >> >
> > >> >
> > >> > On Tue, Feb 21, 2017 at 4:45 PM, Roman Shaposhnik <
> > roman@shaposhnik.org>
> > >> > wrote:
> > >> >
> > >> >> On Tue, Feb 21, 2017 at 2:55 PM, Frank McQuillan
> > >> >> <fm...@pivotal.io>
> > >> >> wrote:
> > >> >> > Agree with Rahul re putting up an RC2 with the suggested changes
> > from
> > >> >> Roman,
> > >> >> > including incorporating Ed's comments on copyright year and top
> > level
> > >> >> folder
> > >> >> > naming. These are really items but let's respond to the RC1
> > >> >> > reviewers
> > >> >> the
> > >> >> > best way we can.
> > >> >>
> > >> >> +1 to a respin.
> > >> >>
> > >> >> > Regarding the ASF legal issue being discussed, MADLib community
> is
> > >> >> > more
> > >> >> than
> > >> >> > happy to respond to any guidance from the fine folks at the ASF
> > >> >> > around
> > >> >> > headers with appropriate licensing verbage. We just need to know
> > >> >> > what
> > >> >> that
> > >> >> > guidance is.
> > >> >>
> > >> >> Well, if you're ok respinning next week I hope to get you a final
> > >> >> answer by then.
> > >> >> Might as well kill two birds with the same RC. Or we can quickly
> do a
> > >> >> follow up
> > >> >> release once the licensing headers dust settles. Up to you guys.
> > >> >>
> > >> >> Thanks,
> > >> >> Roman.
> > >> >>
> > >
> > >
> >
>
Re: [VOTE] MADlib v1.10-rc1
Posted by Frank McQuillan <fm...@pivotal.io>.
OK, so we need to go back and do the comparison from the original code
grant in the fall of 2015 to the current 1.10 release candidate.
On Mon, Feb 27, 2017 at 4:19 PM, Roman Shaposhnik <ro...@shaposhnik.org>
wrote:
> Frank, I'm not sure I understand the question. The criteria needs to hold
> for anything that came in via the initial code ingest compared to how the
> master of your project looks now.
>
> Thanks,
> Roman.
>
> On Mon, Feb 27, 2017 at 4:10 PM, Frank McQuillan <fm...@pivotal.io>
> wrote:
> > Roman,
> >
> > Does this apply retro-actively back to initial grant of the code to
> ASF? Or
> > just from the last release 1.9.1?
> >
> > Frank
> >
> > On Sun, Feb 26, 2017 at 11:23 PM, Roman Shaposhnik <roman@shaposhnik.org
> >
> > wrote:
> >>
> >> Here's the ultimate resolution on the IP issue:
> >> * we don't do anything with existing (BSD) files even if we edit them
> >> * every new file we create gets an ASF license header
> >>
> >> More details:
> >>
> >> https://issues.apache.org/jira/browse/LEGAL-293?
> focusedCommentId=15881595&page=com.atlassian.jira.
> plugin.system.issuetabpanels:comment-tabpanel#comment-15881595
> >>
> >> Thanks,
> >> Roman.
> >>
> >> On Tue, Feb 21, 2017 at 5:54 PM, Frank McQuillan <fmcquillan@pivotal.io
> >
> >> wrote:
> >> > Thanks Roman for working on this.
> >> >
> >> > If you feel a final answer will be ready next week, then yes by all
> >> > means l
> >> > would suggest to the community that we wait and re-spin an RC2 with
> the
> >> > license headers issue resolved. Seems less overhead and effort than a
> >> > quick follow on release right after 1.10. Also, there some momentum
> >> > going
> >> > with the legal discussion, so let's take advantage of that.
> >> >
> >> > Satoshi (release manager), are you OK pausing the RC2 until we hear
> back
> >> > from Roman next week?
> >> >
> >> > Thank you,
> >> > Frank
> >> >
> >> >
> >> > On Tue, Feb 21, 2017 at 4:45 PM, Roman Shaposhnik <
> roman@shaposhnik.org>
> >> > wrote:
> >> >
> >> >> On Tue, Feb 21, 2017 at 2:55 PM, Frank McQuillan
> >> >> <fm...@pivotal.io>
> >> >> wrote:
> >> >> > Agree with Rahul re putting up an RC2 with the suggested changes
> from
> >> >> Roman,
> >> >> > including incorporating Ed's comments on copyright year and top
> level
> >> >> folder
> >> >> > naming. These are really items but let's respond to the RC1
> >> >> > reviewers
> >> >> the
> >> >> > best way we can.
> >> >>
> >> >> +1 to a respin.
> >> >>
> >> >> > Regarding the ASF legal issue being discussed, MADLib community is
> >> >> > more
> >> >> than
> >> >> > happy to respond to any guidance from the fine folks at the ASF
> >> >> > around
> >> >> > headers with appropriate licensing verbage. We just need to know
> >> >> > what
> >> >> that
> >> >> > guidance is.
> >> >>
> >> >> Well, if you're ok respinning next week I hope to get you a final
> >> >> answer by then.
> >> >> Might as well kill two birds with the same RC. Or we can quickly do a
> >> >> follow up
> >> >> release once the licensing headers dust settles. Up to you guys.
> >> >>
> >> >> Thanks,
> >> >> Roman.
> >> >>
> >
> >
>
Re: [VOTE] MADlib v1.10-rc1
Posted by Frank McQuillan <fm...@pivotal.io>.
OK, so we need to go back and do the comparison from the original code
grant in the fall of 2015 to the current 1.10 release candidate.
On Mon, Feb 27, 2017 at 4:19 PM, Roman Shaposhnik <ro...@shaposhnik.org>
wrote:
> Frank, I'm not sure I understand the question. The criteria needs to hold
> for anything that came in via the initial code ingest compared to how the
> master of your project looks now.
>
> Thanks,
> Roman.
>
> On Mon, Feb 27, 2017 at 4:10 PM, Frank McQuillan <fm...@pivotal.io>
> wrote:
> > Roman,
> >
> > Does this apply retro-actively back to initial grant of the code to
> ASF? Or
> > just from the last release 1.9.1?
> >
> > Frank
> >
> > On Sun, Feb 26, 2017 at 11:23 PM, Roman Shaposhnik <roman@shaposhnik.org
> >
> > wrote:
> >>
> >> Here's the ultimate resolution on the IP issue:
> >> * we don't do anything with existing (BSD) files even if we edit them
> >> * every new file we create gets an ASF license header
> >>
> >> More details:
> >>
> >> https://issues.apache.org/jira/browse/LEGAL-293?
> focusedCommentId=15881595&page=com.atlassian.jira.
> plugin.system.issuetabpanels:comment-tabpanel#comment-15881595
> >>
> >> Thanks,
> >> Roman.
> >>
> >> On Tue, Feb 21, 2017 at 5:54 PM, Frank McQuillan <fmcquillan@pivotal.io
> >
> >> wrote:
> >> > Thanks Roman for working on this.
> >> >
> >> > If you feel a final answer will be ready next week, then yes by all
> >> > means l
> >> > would suggest to the community that we wait and re-spin an RC2 with
> the
> >> > license headers issue resolved. Seems less overhead and effort than a
> >> > quick follow on release right after 1.10. Also, there some momentum
> >> > going
> >> > with the legal discussion, so let's take advantage of that.
> >> >
> >> > Satoshi (release manager), are you OK pausing the RC2 until we hear
> back
> >> > from Roman next week?
> >> >
> >> > Thank you,
> >> > Frank
> >> >
> >> >
> >> > On Tue, Feb 21, 2017 at 4:45 PM, Roman Shaposhnik <
> roman@shaposhnik.org>
> >> > wrote:
> >> >
> >> >> On Tue, Feb 21, 2017 at 2:55 PM, Frank McQuillan
> >> >> <fm...@pivotal.io>
> >> >> wrote:
> >> >> > Agree with Rahul re putting up an RC2 with the suggested changes
> from
> >> >> Roman,
> >> >> > including incorporating Ed's comments on copyright year and top
> level
> >> >> folder
> >> >> > naming. These are really items but let's respond to the RC1
> >> >> > reviewers
> >> >> the
> >> >> > best way we can.
> >> >>
> >> >> +1 to a respin.
> >> >>
> >> >> > Regarding the ASF legal issue being discussed, MADLib community is
> >> >> > more
> >> >> than
> >> >> > happy to respond to any guidance from the fine folks at the ASF
> >> >> > around
> >> >> > headers with appropriate licensing verbage. We just need to know
> >> >> > what
> >> >> that
> >> >> > guidance is.
> >> >>
> >> >> Well, if you're ok respinning next week I hope to get you a final
> >> >> answer by then.
> >> >> Might as well kill two birds with the same RC. Or we can quickly do a
> >> >> follow up
> >> >> release once the licensing headers dust settles. Up to you guys.
> >> >>
> >> >> Thanks,
> >> >> Roman.
> >> >>
> >
> >
>
Re: [VOTE] MADlib v1.10-rc1
Posted by Roman Shaposhnik <ro...@shaposhnik.org>.
Frank, I'm not sure I understand the question. The criteria needs to hold
for anything that came in via the initial code ingest compared to how the
master of your project looks now.
Thanks,
Roman.
On Mon, Feb 27, 2017 at 4:10 PM, Frank McQuillan <fm...@pivotal.io> wrote:
> Roman,
>
> Does this apply retro-actively back to initial grant of the code to ASF? Or
> just from the last release 1.9.1?
>
> Frank
>
> On Sun, Feb 26, 2017 at 11:23 PM, Roman Shaposhnik <ro...@shaposhnik.org>
> wrote:
>>
>> Here's the ultimate resolution on the IP issue:
>> * we don't do anything with existing (BSD) files even if we edit them
>> * every new file we create gets an ASF license header
>>
>> More details:
>>
>> https://issues.apache.org/jira/browse/LEGAL-293?focusedCommentId=15881595&page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel#comment-15881595
>>
>> Thanks,
>> Roman.
>>
>> On Tue, Feb 21, 2017 at 5:54 PM, Frank McQuillan <fm...@pivotal.io>
>> wrote:
>> > Thanks Roman for working on this.
>> >
>> > If you feel a final answer will be ready next week, then yes by all
>> > means l
>> > would suggest to the community that we wait and re-spin an RC2 with the
>> > license headers issue resolved. Seems less overhead and effort than a
>> > quick follow on release right after 1.10. Also, there some momentum
>> > going
>> > with the legal discussion, so let's take advantage of that.
>> >
>> > Satoshi (release manager), are you OK pausing the RC2 until we hear back
>> > from Roman next week?
>> >
>> > Thank you,
>> > Frank
>> >
>> >
>> > On Tue, Feb 21, 2017 at 4:45 PM, Roman Shaposhnik <ro...@shaposhnik.org>
>> > wrote:
>> >
>> >> On Tue, Feb 21, 2017 at 2:55 PM, Frank McQuillan
>> >> <fm...@pivotal.io>
>> >> wrote:
>> >> > Agree with Rahul re putting up an RC2 with the suggested changes from
>> >> Roman,
>> >> > including incorporating Ed's comments on copyright year and top level
>> >> folder
>> >> > naming. These are really items but let's respond to the RC1
>> >> > reviewers
>> >> the
>> >> > best way we can.
>> >>
>> >> +1 to a respin.
>> >>
>> >> > Regarding the ASF legal issue being discussed, MADLib community is
>> >> > more
>> >> than
>> >> > happy to respond to any guidance from the fine folks at the ASF
>> >> > around
>> >> > headers with appropriate licensing verbage. We just need to know
>> >> > what
>> >> that
>> >> > guidance is.
>> >>
>> >> Well, if you're ok respinning next week I hope to get you a final
>> >> answer by then.
>> >> Might as well kill two birds with the same RC. Or we can quickly do a
>> >> follow up
>> >> release once the licensing headers dust settles. Up to you guys.
>> >>
>> >> Thanks,
>> >> Roman.
>> >>
>
>
Re: [VOTE] MADlib v1.10-rc1
Posted by Roman Shaposhnik <ro...@shaposhnik.org>.
Frank, I'm not sure I understand the question. The criteria needs to hold
for anything that came in via the initial code ingest compared to how the
master of your project looks now.
Thanks,
Roman.
On Mon, Feb 27, 2017 at 4:10 PM, Frank McQuillan <fm...@pivotal.io> wrote:
> Roman,
>
> Does this apply retro-actively back to initial grant of the code to ASF? Or
> just from the last release 1.9.1?
>
> Frank
>
> On Sun, Feb 26, 2017 at 11:23 PM, Roman Shaposhnik <ro...@shaposhnik.org>
> wrote:
>>
>> Here's the ultimate resolution on the IP issue:
>> * we don't do anything with existing (BSD) files even if we edit them
>> * every new file we create gets an ASF license header
>>
>> More details:
>>
>> https://issues.apache.org/jira/browse/LEGAL-293?focusedCommentId=15881595&page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel#comment-15881595
>>
>> Thanks,
>> Roman.
>>
>> On Tue, Feb 21, 2017 at 5:54 PM, Frank McQuillan <fm...@pivotal.io>
>> wrote:
>> > Thanks Roman for working on this.
>> >
>> > If you feel a final answer will be ready next week, then yes by all
>> > means l
>> > would suggest to the community that we wait and re-spin an RC2 with the
>> > license headers issue resolved. Seems less overhead and effort than a
>> > quick follow on release right after 1.10. Also, there some momentum
>> > going
>> > with the legal discussion, so let's take advantage of that.
>> >
>> > Satoshi (release manager), are you OK pausing the RC2 until we hear back
>> > from Roman next week?
>> >
>> > Thank you,
>> > Frank
>> >
>> >
>> > On Tue, Feb 21, 2017 at 4:45 PM, Roman Shaposhnik <ro...@shaposhnik.org>
>> > wrote:
>> >
>> >> On Tue, Feb 21, 2017 at 2:55 PM, Frank McQuillan
>> >> <fm...@pivotal.io>
>> >> wrote:
>> >> > Agree with Rahul re putting up an RC2 with the suggested changes from
>> >> Roman,
>> >> > including incorporating Ed's comments on copyright year and top level
>> >> folder
>> >> > naming. These are really items but let's respond to the RC1
>> >> > reviewers
>> >> the
>> >> > best way we can.
>> >>
>> >> +1 to a respin.
>> >>
>> >> > Regarding the ASF legal issue being discussed, MADLib community is
>> >> > more
>> >> than
>> >> > happy to respond to any guidance from the fine folks at the ASF
>> >> > around
>> >> > headers with appropriate licensing verbage. We just need to know
>> >> > what
>> >> that
>> >> > guidance is.
>> >>
>> >> Well, if you're ok respinning next week I hope to get you a final
>> >> answer by then.
>> >> Might as well kill two birds with the same RC. Or we can quickly do a
>> >> follow up
>> >> release once the licensing headers dust settles. Up to you guys.
>> >>
>> >> Thanks,
>> >> Roman.
>> >>
>
>
Re: [VOTE] MADlib v1.10-rc1
Posted by Frank McQuillan <fm...@pivotal.io>.
Roman,
Does this apply retro-actively back to initial grant of the code to ASF?
Or just from the last release 1.9.1?
Frank
On Sun, Feb 26, 2017 at 11:23 PM, Roman Shaposhnik <ro...@shaposhnik.org>
wrote:
> Here's the ultimate resolution on the IP issue:
> * we don't do anything with existing (BSD) files even if we edit them
> * every new file we create gets an ASF license header
>
> More details:
> https://issues.apache.org/jira/browse/LEGAL-293?
> focusedCommentId=15881595&page=com.atlassian.jira.
> plugin.system.issuetabpanels:comment-tabpanel#comment-15881595
>
> Thanks,
> Roman.
>
> On Tue, Feb 21, 2017 at 5:54 PM, Frank McQuillan <fm...@pivotal.io>
> wrote:
> > Thanks Roman for working on this.
> >
> > If you feel a final answer will be ready next week, then yes by all
> means l
> > would suggest to the community that we wait and re-spin an RC2 with the
> > license headers issue resolved. Seems less overhead and effort than a
> > quick follow on release right after 1.10. Also, there some momentum
> going
> > with the legal discussion, so let's take advantage of that.
> >
> > Satoshi (release manager), are you OK pausing the RC2 until we hear back
> > from Roman next week?
> >
> > Thank you,
> > Frank
> >
> >
> > On Tue, Feb 21, 2017 at 4:45 PM, Roman Shaposhnik <ro...@shaposhnik.org>
> > wrote:
> >
> >> On Tue, Feb 21, 2017 at 2:55 PM, Frank McQuillan <fmcquillan@pivotal.io
> >
> >> wrote:
> >> > Agree with Rahul re putting up an RC2 with the suggested changes from
> >> Roman,
> >> > including incorporating Ed's comments on copyright year and top level
> >> folder
> >> > naming. These are really items but let's respond to the RC1 reviewers
> >> the
> >> > best way we can.
> >>
> >> +1 to a respin.
> >>
> >> > Regarding the ASF legal issue being discussed, MADLib community is
> more
> >> than
> >> > happy to respond to any guidance from the fine folks at the ASF around
> >> > headers with appropriate licensing verbage. We just need to know what
> >> that
> >> > guidance is.
> >>
> >> Well, if you're ok respinning next week I hope to get you a final
> >> answer by then.
> >> Might as well kill two birds with the same RC. Or we can quickly do a
> >> follow up
> >> release once the licensing headers dust settles. Up to you guys.
> >>
> >> Thanks,
> >> Roman.
> >>
>
Re: [VOTE] MADlib v1.10-rc1
Posted by Frank McQuillan <fm...@pivotal.io>.
Roman,
Does this apply retro-actively back to initial grant of the code to ASF?
Or just from the last release 1.9.1?
Frank
On Sun, Feb 26, 2017 at 11:23 PM, Roman Shaposhnik <ro...@shaposhnik.org>
wrote:
> Here's the ultimate resolution on the IP issue:
> * we don't do anything with existing (BSD) files even if we edit them
> * every new file we create gets an ASF license header
>
> More details:
> https://issues.apache.org/jira/browse/LEGAL-293?
> focusedCommentId=15881595&page=com.atlassian.jira.
> plugin.system.issuetabpanels:comment-tabpanel#comment-15881595
>
> Thanks,
> Roman.
>
> On Tue, Feb 21, 2017 at 5:54 PM, Frank McQuillan <fm...@pivotal.io>
> wrote:
> > Thanks Roman for working on this.
> >
> > If you feel a final answer will be ready next week, then yes by all
> means l
> > would suggest to the community that we wait and re-spin an RC2 with the
> > license headers issue resolved. Seems less overhead and effort than a
> > quick follow on release right after 1.10. Also, there some momentum
> going
> > with the legal discussion, so let's take advantage of that.
> >
> > Satoshi (release manager), are you OK pausing the RC2 until we hear back
> > from Roman next week?
> >
> > Thank you,
> > Frank
> >
> >
> > On Tue, Feb 21, 2017 at 4:45 PM, Roman Shaposhnik <ro...@shaposhnik.org>
> > wrote:
> >
> >> On Tue, Feb 21, 2017 at 2:55 PM, Frank McQuillan <fmcquillan@pivotal.io
> >
> >> wrote:
> >> > Agree with Rahul re putting up an RC2 with the suggested changes from
> >> Roman,
> >> > including incorporating Ed's comments on copyright year and top level
> >> folder
> >> > naming. These are really items but let's respond to the RC1 reviewers
> >> the
> >> > best way we can.
> >>
> >> +1 to a respin.
> >>
> >> > Regarding the ASF legal issue being discussed, MADLib community is
> more
> >> than
> >> > happy to respond to any guidance from the fine folks at the ASF around
> >> > headers with appropriate licensing verbage. We just need to know what
> >> that
> >> > guidance is.
> >>
> >> Well, if you're ok respinning next week I hope to get you a final
> >> answer by then.
> >> Might as well kill two birds with the same RC. Or we can quickly do a
> >> follow up
> >> release once the licensing headers dust settles. Up to you guys.
> >>
> >> Thanks,
> >> Roman.
> >>
>
Re: [VOTE] MADlib v1.10-rc1
Posted by Roman Shaposhnik <ro...@shaposhnik.org>.
Here's the ultimate resolution on the IP issue:
* we don't do anything with existing (BSD) files even if we edit them
* every new file we create gets an ASF license header
More details:
https://issues.apache.org/jira/browse/LEGAL-293?focusedCommentId=15881595&page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel#comment-15881595
Thanks,
Roman.
On Tue, Feb 21, 2017 at 5:54 PM, Frank McQuillan <fm...@pivotal.io> wrote:
> Thanks Roman for working on this.
>
> If you feel a final answer will be ready next week, then yes by all means l
> would suggest to the community that we wait and re-spin an RC2 with the
> license headers issue resolved. Seems less overhead and effort than a
> quick follow on release right after 1.10. Also, there some momentum going
> with the legal discussion, so let's take advantage of that.
>
> Satoshi (release manager), are you OK pausing the RC2 until we hear back
> from Roman next week?
>
> Thank you,
> Frank
>
>
> On Tue, Feb 21, 2017 at 4:45 PM, Roman Shaposhnik <ro...@shaposhnik.org>
> wrote:
>
>> On Tue, Feb 21, 2017 at 2:55 PM, Frank McQuillan <fm...@pivotal.io>
>> wrote:
>> > Agree with Rahul re putting up an RC2 with the suggested changes from
>> Roman,
>> > including incorporating Ed's comments on copyright year and top level
>> folder
>> > naming. These are really items but let's respond to the RC1 reviewers
>> the
>> > best way we can.
>>
>> +1 to a respin.
>>
>> > Regarding the ASF legal issue being discussed, MADLib community is more
>> than
>> > happy to respond to any guidance from the fine folks at the ASF around
>> > headers with appropriate licensing verbage. We just need to know what
>> that
>> > guidance is.
>>
>> Well, if you're ok respinning next week I hope to get you a final
>> answer by then.
>> Might as well kill two birds with the same RC. Or we can quickly do a
>> follow up
>> release once the licensing headers dust settles. Up to you guys.
>>
>> Thanks,
>> Roman.
>>
Re: [VOTE] MADlib v1.10-rc1
Posted by Roman Shaposhnik <ro...@shaposhnik.org>.
Here's the ultimate resolution on the IP issue:
* we don't do anything with existing (BSD) files even if we edit them
* every new file we create gets an ASF license header
More details:
https://issues.apache.org/jira/browse/LEGAL-293?focusedCommentId=15881595&page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel#comment-15881595
Thanks,
Roman.
On Tue, Feb 21, 2017 at 5:54 PM, Frank McQuillan <fm...@pivotal.io> wrote:
> Thanks Roman for working on this.
>
> If you feel a final answer will be ready next week, then yes by all means l
> would suggest to the community that we wait and re-spin an RC2 with the
> license headers issue resolved. Seems less overhead and effort than a
> quick follow on release right after 1.10. Also, there some momentum going
> with the legal discussion, so let's take advantage of that.
>
> Satoshi (release manager), are you OK pausing the RC2 until we hear back
> from Roman next week?
>
> Thank you,
> Frank
>
>
> On Tue, Feb 21, 2017 at 4:45 PM, Roman Shaposhnik <ro...@shaposhnik.org>
> wrote:
>
>> On Tue, Feb 21, 2017 at 2:55 PM, Frank McQuillan <fm...@pivotal.io>
>> wrote:
>> > Agree with Rahul re putting up an RC2 with the suggested changes from
>> Roman,
>> > including incorporating Ed's comments on copyright year and top level
>> folder
>> > naming. These are really items but let's respond to the RC1 reviewers
>> the
>> > best way we can.
>>
>> +1 to a respin.
>>
>> > Regarding the ASF legal issue being discussed, MADLib community is more
>> than
>> > happy to respond to any guidance from the fine folks at the ASF around
>> > headers with appropriate licensing verbage. We just need to know what
>> that
>> > guidance is.
>>
>> Well, if you're ok respinning next week I hope to get you a final
>> answer by then.
>> Might as well kill two birds with the same RC. Or we can quickly do a
>> follow up
>> release once the licensing headers dust settles. Up to you guys.
>>
>> Thanks,
>> Roman.
>>
Re: [VOTE] MADlib v1.10-rc1
Posted by Frank McQuillan <fm...@pivotal.io>.
Thanks Roman for working on this.
If you feel a final answer will be ready next week, then yes by all means l
would suggest to the community that we wait and re-spin an RC2 with the
license headers issue resolved. Seems less overhead and effort than a
quick follow on release right after 1.10. Also, there some momentum going
with the legal discussion, so let's take advantage of that.
Satoshi (release manager), are you OK pausing the RC2 until we hear back
from Roman next week?
Thank you,
Frank
On Tue, Feb 21, 2017 at 4:45 PM, Roman Shaposhnik <ro...@shaposhnik.org>
wrote:
> On Tue, Feb 21, 2017 at 2:55 PM, Frank McQuillan <fm...@pivotal.io>
> wrote:
> > Agree with Rahul re putting up an RC2 with the suggested changes from
> Roman,
> > including incorporating Ed's comments on copyright year and top level
> folder
> > naming. These are really items but let's respond to the RC1 reviewers
> the
> > best way we can.
>
> +1 to a respin.
>
> > Regarding the ASF legal issue being discussed, MADLib community is more
> than
> > happy to respond to any guidance from the fine folks at the ASF around
> > headers with appropriate licensing verbage. We just need to know what
> that
> > guidance is.
>
> Well, if you're ok respinning next week I hope to get you a final
> answer by then.
> Might as well kill two birds with the same RC. Or we can quickly do a
> follow up
> release once the licensing headers dust settles. Up to you guys.
>
> Thanks,
> Roman.
>
Re: [VOTE] MADlib v1.10-rc1
Posted by Frank McQuillan <fm...@pivotal.io>.
Thanks Roman for working on this.
If you feel a final answer will be ready next week, then yes by all means l
would suggest to the community that we wait and re-spin an RC2 with the
license headers issue resolved. Seems less overhead and effort than a
quick follow on release right after 1.10. Also, there some momentum going
with the legal discussion, so let's take advantage of that.
Satoshi (release manager), are you OK pausing the RC2 until we hear back
from Roman next week?
Thank you,
Frank
On Tue, Feb 21, 2017 at 4:45 PM, Roman Shaposhnik <ro...@shaposhnik.org>
wrote:
> On Tue, Feb 21, 2017 at 2:55 PM, Frank McQuillan <fm...@pivotal.io>
> wrote:
> > Agree with Rahul re putting up an RC2 with the suggested changes from
> Roman,
> > including incorporating Ed's comments on copyright year and top level
> folder
> > naming. These are really items but let's respond to the RC1 reviewers
> the
> > best way we can.
>
> +1 to a respin.
>
> > Regarding the ASF legal issue being discussed, MADLib community is more
> than
> > happy to respond to any guidance from the fine folks at the ASF around
> > headers with appropriate licensing verbage. We just need to know what
> that
> > guidance is.
>
> Well, if you're ok respinning next week I hope to get you a final
> answer by then.
> Might as well kill two birds with the same RC. Or we can quickly do a
> follow up
> release once the licensing headers dust settles. Up to you guys.
>
> Thanks,
> Roman.
>
Re: [VOTE] MADlib v1.10-rc1
Posted by Roman Shaposhnik <ro...@shaposhnik.org>.
On Tue, Feb 21, 2017 at 2:55 PM, Frank McQuillan <fm...@pivotal.io> wrote:
> Agree with Rahul re putting up an RC2 with the suggested changes from Roman,
> including incorporating Ed's comments on copyright year and top level folder
> naming. These are really items but let's respond to the RC1 reviewers the
> best way we can.
+1 to a respin.
> Regarding the ASF legal issue being discussed, MADLib community is more than
> happy to respond to any guidance from the fine folks at the ASF around
> headers with appropriate licensing verbage. We just need to know what that
> guidance is.
Well, if you're ok respinning next week I hope to get you a final
answer by then.
Might as well kill two birds with the same RC. Or we can quickly do a follow up
release once the licensing headers dust settles. Up to you guys.
Thanks,
Roman.
Re: [VOTE] MADlib v1.10-rc1
Posted by Roman Shaposhnik <ro...@shaposhnik.org>.
On Tue, Feb 21, 2017 at 2:55 PM, Frank McQuillan <fm...@pivotal.io> wrote:
> Agree with Rahul re putting up an RC2 with the suggested changes from Roman,
> including incorporating Ed's comments on copyright year and top level folder
> naming. These are really items but let's respond to the RC1 reviewers the
> best way we can.
+1 to a respin.
> Regarding the ASF legal issue being discussed, MADLib community is more than
> happy to respond to any guidance from the fine folks at the ASF around
> headers with appropriate licensing verbage. We just need to know what that
> guidance is.
Well, if you're ok respinning next week I hope to get you a final
answer by then.
Might as well kill two birds with the same RC. Or we can quickly do a follow up
release once the licensing headers dust settles. Up to you guys.
Thanks,
Roman.
Re: [VOTE] MADlib v1.10-rc1
Posted by Frank McQuillan <fm...@pivotal.io>.
Agree with Rahul re putting up an RC2 with the suggested changes from
Roman, including incorporating Ed's comments on copyright year and top
level folder naming. These are really items but let's respond to the RC1
reviewers the best way we can.
Regarding the ASF legal issue being discussed, MADLib community is more
than happy to respond to any guidance from the fine folks at the ASF around
headers with appropriate licensing verbage. We just need to know what that
guidance is.
Frank
On Tue, Feb 21, 2017 at 10:58 AM, Ed Espino <es...@apache.org> wrote:
> Orhan,
>
> One more mildly interesting thing I noticed, I can't use the gsha512sum and
> gmd5sum commands to validate the corresponding checksum values easily. Even
> though the commands (see below links) available on the Apache site do
> generate correct checksums, they aren't conducive to a quick validation of
> them.
>
>
> - What Is An MD5 Checksum?
> <https://www.apache.org/dev/release-signing.html#md5>
> - What is a SHA checksum?
> <https://www.apache.org/dev/release-signing.html#sha-checksum>
>
> For reference, here are the processes and validation steps we use for the
> Apache HAWQ incubator project. They may be of some use to your project and
> help those validating checksums.
>
> - Create the Release Candidate
> <https://cwiki.apache.org/confluence/display/HAWQ/
> Release+Process%3A+Step+by+step+guide#ReleaseProcess:Stepbystepguide-
> CreatetheReleaseCandidate>
>
> Hope this helps,
> -=e
>
> On Thu, Feb 16, 2017 at 12:06 PM, Orhan Kislal <ok...@pivotal.io> wrote:
>
> > Hi Ed,
> >
> > Thanks for the review. One of the comments from the previous release was
> a
> > preference towards a signature with an Apache id. Since Satoshi-san is
> not
> > an Apache committer yet, I took care of the signing process.
> >
> > Thanks,
> >
> > Orhan Kislal
> >
> > On Thu, Feb 16, 2017 at 11:58 AM, Ed Espino <es...@apache.org> wrote:
> >
> > > A few MADlib v1.10-rc1 observations from a HAWQ incubator committer.
> > >
> > > - The Copyright year (2016) in the NOTICE file needs to be updated
> to
> > > 2017. I believe this can be handled in next release.
> > > - As it still applies, similar to a past comment by Roman ([VOTE]
> > MADlib
> > > v1.9.1-rc2
> > > <https://lists.apache.org/thread.html/
> 981b4c24eaa2ab069b8e18f7aa4bdd
> > > c7a78d3a9dc26bf659af94fcfe@%3Cgeneral.incubator.apache.org%3E>)
> > > - *"* name of the top level folder in the archive is weird. The
> usual
> > > practice is to call the top level folder as <project name>-<version
> > > ID>*"*
> > > (example: *apache-madlib-src-1.10-incubating* instead of
> > > *incubator-madlib*)
> > > - I'm more curious than anything. Why did Orhan sign the release? I
> > was
> > > expecting the release manager (Satoshi Nagayasu) to have signed the
> > > release.
> > > - Checksums and PGP signature are good.
> > > - ASF headers check: I spot checked files added (git whatchanged
> > > --diff-filter=A) since the last release. ASF headers look good.
> Nice
> > > Job!
> > >
> > > I was going to try and build but I ran past my allotted time limit for
> > this
> > > review. Hopefully, I can try this soon.
> > >
> > > Regards,
> > > -=ed espino
> > >
> > > On Thu, Feb 16, 2017 at 10:05 AM, Orhan Kislal <ok...@pivotal.io>
> > wrote:
> > >
> > > > +1
> > > >
> > > > Orhan Kislal
> > > >
> > > > On Thu, Feb 16, 2017 at 9:23 AM, Joe Hellerstein <
> > > hellerstein@berkeley.edu
> > > > >
> > > > wrote:
> > > >
> > > > > +1
> > > > >
> > > > > Sent from a telephone.
> > > > >
> > > > > > On Feb 16, 2017, at 9:17 AM, Frank McQuillan <
> > fmcquillan@pivotal.io>
> > > > > wrote:
> > > > > >
> > > > > > +1
> > > > > >
> > > > > > Frank McQuillan
> > > > > >
> > > > > >> On Wed, Feb 15, 2017 at 7:27 PM, Satoshi Nagayasu <
> > snaga@uptime.jp>
> > > > > wrote:
> > > > > >>
> > > > > >> Hello MADlib community,
> > > > > >>
> > > > > >> We have created a MADlib 1.10 RC-1, with the artifacts below up
> > for
> > > a
> > > > > vote.
> > > > > >>
> > > > > >> This will be the 4th release for Apache MADlib (incubating).
> > > > > >>
> > > > > >> The main goals of this release are:
> > > > > >> * new modules (single source shortest path for graph analytics,
> > > encode
> > > > > >> categorical variables, K-nearest neighbors)
> > > > > >> * improvements to existing modules (add grouping support to
> > elastic
> > > > > >> net and PCA, add cross validation to elastic net, array input
> for
> > > > > >> K-means, verbose output option for DT and RF, limit itemset size
> > in
> > > > > >> association rules, various madpack installer improvements)
> > > > > >> * platform updates (PostgreSQL 9.6)
> > > > > >> * bug fixes
> > > > > >> * doc improvements
> > > > > >>
> > > > > >> For more information including release notes, please see:
> > > > > >> https://cwiki.apache.org/confluence/display/MADLIB/MADlib+1.10
> > > > > >>
> > > > > >> *** Please download, review and vote by Sat Feb 18, 2017 @ 6pm
> PST
> > > ***
> > > > > >>
> > > > > >> We're voting upon the source (tag): rc/1.10.0-rc1
> > > > > >> https://github.com/apache/incubator-madlib/tree/rc/1.10.0-rc1
> > > > > >>
> > > > > >> Source Files:
> > > > > >> https://dist.apache.org/repos/dist/dev/incubator/madlib/1.
> > > > > >> 10.0-incubating-rc1/
> > > > > >>
> > > > > >> Commit to be voted upon:
> > > > > >> https://github.com/apache/incubator-madlib/commit/
> > > > > >> ea17530bfe22a1fde173d7fa83508cbcd9924c20
> > > > > >>
> > > > > >> KEYS file containing PGP Keys we use to sign the release:
> > > > > >> https://dist.apache.org/repos/dist/dev/incubator/madlib/KEYS
> > > > > >>
> > > > > >> To help in tallying the vote, can PMC members please be sure to
> > > > > >> indicate "(binding)" with their vote.
> > > > > >>
> > > > > >> [ ] +1 approve
> > > > > >> [ ] +0 no opinion
> > > > > >> [ ] -1 disapprove (and reason why)
> > > > > >>
> > > > > >> --
> > > > > >> Satoshi Nagayasu <sn...@uptime.jp>
> > > > > >>
> > > > >
> > > >
> > >
> > >
> > >
> > > --
> > > *Ed Espino*
> > > *espino@apache.org <es...@apache.org>*
> > >
> >
>
>
>
> --
> *Ed Espino*
> *espino@apache.org <es...@apache.org>*
>
Re: [VOTE] MADlib v1.10-rc1
Posted by Frank McQuillan <fm...@pivotal.io>.
Agree with Rahul re putting up an RC2 with the suggested changes from
Roman, including incorporating Ed's comments on copyright year and top
level folder naming. These are really items but let's respond to the RC1
reviewers the best way we can.
Regarding the ASF legal issue being discussed, MADLib community is more
than happy to respond to any guidance from the fine folks at the ASF around
headers with appropriate licensing verbage. We just need to know what that
guidance is.
Frank
On Tue, Feb 21, 2017 at 10:58 AM, Ed Espino <es...@apache.org> wrote:
> Orhan,
>
> One more mildly interesting thing I noticed, I can't use the gsha512sum and
> gmd5sum commands to validate the corresponding checksum values easily. Even
> though the commands (see below links) available on the Apache site do
> generate correct checksums, they aren't conducive to a quick validation of
> them.
>
>
> - What Is An MD5 Checksum?
> <https://www.apache.org/dev/release-signing.html#md5>
> - What is a SHA checksum?
> <https://www.apache.org/dev/release-signing.html#sha-checksum>
>
> For reference, here are the processes and validation steps we use for the
> Apache HAWQ incubator project. They may be of some use to your project and
> help those validating checksums.
>
> - Create the Release Candidate
> <https://cwiki.apache.org/confluence/display/HAWQ/
> Release+Process%3A+Step+by+step+guide#ReleaseProcess:Stepbystepguide-
> CreatetheReleaseCandidate>
>
> Hope this helps,
> -=e
>
> On Thu, Feb 16, 2017 at 12:06 PM, Orhan Kislal <ok...@pivotal.io> wrote:
>
> > Hi Ed,
> >
> > Thanks for the review. One of the comments from the previous release was
> a
> > preference towards a signature with an Apache id. Since Satoshi-san is
> not
> > an Apache committer yet, I took care of the signing process.
> >
> > Thanks,
> >
> > Orhan Kislal
> >
> > On Thu, Feb 16, 2017 at 11:58 AM, Ed Espino <es...@apache.org> wrote:
> >
> > > A few MADlib v1.10-rc1 observations from a HAWQ incubator committer.
> > >
> > > - The Copyright year (2016) in the NOTICE file needs to be updated
> to
> > > 2017. I believe this can be handled in next release.
> > > - As it still applies, similar to a past comment by Roman ([VOTE]
> > MADlib
> > > v1.9.1-rc2
> > > <https://lists.apache.org/thread.html/
> 981b4c24eaa2ab069b8e18f7aa4bdd
> > > c7a78d3a9dc26bf659af94fcfe@%3Cgeneral.incubator.apache.org%3E>)
> > > - *"* name of the top level folder in the archive is weird. The
> usual
> > > practice is to call the top level folder as <project name>-<version
> > > ID>*"*
> > > (example: *apache-madlib-src-1.10-incubating* instead of
> > > *incubator-madlib*)
> > > - I'm more curious than anything. Why did Orhan sign the release? I
> > was
> > > expecting the release manager (Satoshi Nagayasu) to have signed the
> > > release.
> > > - Checksums and PGP signature are good.
> > > - ASF headers check: I spot checked files added (git whatchanged
> > > --diff-filter=A) since the last release. ASF headers look good.
> Nice
> > > Job!
> > >
> > > I was going to try and build but I ran past my allotted time limit for
> > this
> > > review. Hopefully, I can try this soon.
> > >
> > > Regards,
> > > -=ed espino
> > >
> > > On Thu, Feb 16, 2017 at 10:05 AM, Orhan Kislal <ok...@pivotal.io>
> > wrote:
> > >
> > > > +1
> > > >
> > > > Orhan Kislal
> > > >
> > > > On Thu, Feb 16, 2017 at 9:23 AM, Joe Hellerstein <
> > > hellerstein@berkeley.edu
> > > > >
> > > > wrote:
> > > >
> > > > > +1
> > > > >
> > > > > Sent from a telephone.
> > > > >
> > > > > > On Feb 16, 2017, at 9:17 AM, Frank McQuillan <
> > fmcquillan@pivotal.io>
> > > > > wrote:
> > > > > >
> > > > > > +1
> > > > > >
> > > > > > Frank McQuillan
> > > > > >
> > > > > >> On Wed, Feb 15, 2017 at 7:27 PM, Satoshi Nagayasu <
> > snaga@uptime.jp>
> > > > > wrote:
> > > > > >>
> > > > > >> Hello MADlib community,
> > > > > >>
> > > > > >> We have created a MADlib 1.10 RC-1, with the artifacts below up
> > for
> > > a
> > > > > vote.
> > > > > >>
> > > > > >> This will be the 4th release for Apache MADlib (incubating).
> > > > > >>
> > > > > >> The main goals of this release are:
> > > > > >> * new modules (single source shortest path for graph analytics,
> > > encode
> > > > > >> categorical variables, K-nearest neighbors)
> > > > > >> * improvements to existing modules (add grouping support to
> > elastic
> > > > > >> net and PCA, add cross validation to elastic net, array input
> for
> > > > > >> K-means, verbose output option for DT and RF, limit itemset size
> > in
> > > > > >> association rules, various madpack installer improvements)
> > > > > >> * platform updates (PostgreSQL 9.6)
> > > > > >> * bug fixes
> > > > > >> * doc improvements
> > > > > >>
> > > > > >> For more information including release notes, please see:
> > > > > >> https://cwiki.apache.org/confluence/display/MADLIB/MADlib+1.10
> > > > > >>
> > > > > >> *** Please download, review and vote by Sat Feb 18, 2017 @ 6pm
> PST
> > > ***
> > > > > >>
> > > > > >> We're voting upon the source (tag): rc/1.10.0-rc1
> > > > > >> https://github.com/apache/incubator-madlib/tree/rc/1.10.0-rc1
> > > > > >>
> > > > > >> Source Files:
> > > > > >> https://dist.apache.org/repos/dist/dev/incubator/madlib/1.
> > > > > >> 10.0-incubating-rc1/
> > > > > >>
> > > > > >> Commit to be voted upon:
> > > > > >> https://github.com/apache/incubator-madlib/commit/
> > > > > >> ea17530bfe22a1fde173d7fa83508cbcd9924c20
> > > > > >>
> > > > > >> KEYS file containing PGP Keys we use to sign the release:
> > > > > >> https://dist.apache.org/repos/dist/dev/incubator/madlib/KEYS
> > > > > >>
> > > > > >> To help in tallying the vote, can PMC members please be sure to
> > > > > >> indicate "(binding)" with their vote.
> > > > > >>
> > > > > >> [ ] +1 approve
> > > > > >> [ ] +0 no opinion
> > > > > >> [ ] -1 disapprove (and reason why)
> > > > > >>
> > > > > >> --
> > > > > >> Satoshi Nagayasu <sn...@uptime.jp>
> > > > > >>
> > > > >
> > > >
> > >
> > >
> > >
> > > --
> > > *Ed Espino*
> > > *espino@apache.org <es...@apache.org>*
> > >
> >
>
>
>
> --
> *Ed Espino*
> *espino@apache.org <es...@apache.org>*
>
Re: [VOTE] MADlib v1.10-rc1
Posted by Ed Espino <es...@apache.org>.
Orhan,
One more mildly interesting thing I noticed, I can't use the gsha512sum and
gmd5sum commands to validate the corresponding checksum values easily. Even
though the commands (see below links) available on the Apache site do
generate correct checksums, they aren't conducive to a quick validation of
them.
- What Is An MD5 Checksum?
<https://www.apache.org/dev/release-signing.html#md5>
- What is a SHA checksum?
<https://www.apache.org/dev/release-signing.html#sha-checksum>
For reference, here are the processes and validation steps we use for the
Apache HAWQ incubator project. They may be of some use to your project and
help those validating checksums.
- Create the Release Candidate
<https://cwiki.apache.org/confluence/display/HAWQ/Release+Process%3A+Step+by+step+guide#ReleaseProcess:Stepbystepguide-CreatetheReleaseCandidate>
Hope this helps,
-=e
On Thu, Feb 16, 2017 at 12:06 PM, Orhan Kislal <ok...@pivotal.io> wrote:
> Hi Ed,
>
> Thanks for the review. One of the comments from the previous release was a
> preference towards a signature with an Apache id. Since Satoshi-san is not
> an Apache committer yet, I took care of the signing process.
>
> Thanks,
>
> Orhan Kislal
>
> On Thu, Feb 16, 2017 at 11:58 AM, Ed Espino <es...@apache.org> wrote:
>
> > A few MADlib v1.10-rc1 observations from a HAWQ incubator committer.
> >
> > - The Copyright year (2016) in the NOTICE file needs to be updated to
> > 2017. I believe this can be handled in next release.
> > - As it still applies, similar to a past comment by Roman ([VOTE]
> MADlib
> > v1.9.1-rc2
> > <https://lists.apache.org/thread.html/981b4c24eaa2ab069b8e18f7aa4bdd
> > c7a78d3a9dc26bf659af94fcfe@%3Cgeneral.incubator.apache.org%3E>)
> > - *"* name of the top level folder in the archive is weird. The usual
> > practice is to call the top level folder as <project name>-<version
> > ID>*"*
> > (example: *apache-madlib-src-1.10-incubating* instead of
> > *incubator-madlib*)
> > - I'm more curious than anything. Why did Orhan sign the release? I
> was
> > expecting the release manager (Satoshi Nagayasu) to have signed the
> > release.
> > - Checksums and PGP signature are good.
> > - ASF headers check: I spot checked files added (git whatchanged
> > --diff-filter=A) since the last release. ASF headers look good. Nice
> > Job!
> >
> > I was going to try and build but I ran past my allotted time limit for
> this
> > review. Hopefully, I can try this soon.
> >
> > Regards,
> > -=ed espino
> >
> > On Thu, Feb 16, 2017 at 10:05 AM, Orhan Kislal <ok...@pivotal.io>
> wrote:
> >
> > > +1
> > >
> > > Orhan Kislal
> > >
> > > On Thu, Feb 16, 2017 at 9:23 AM, Joe Hellerstein <
> > hellerstein@berkeley.edu
> > > >
> > > wrote:
> > >
> > > > +1
> > > >
> > > > Sent from a telephone.
> > > >
> > > > > On Feb 16, 2017, at 9:17 AM, Frank McQuillan <
> fmcquillan@pivotal.io>
> > > > wrote:
> > > > >
> > > > > +1
> > > > >
> > > > > Frank McQuillan
> > > > >
> > > > >> On Wed, Feb 15, 2017 at 7:27 PM, Satoshi Nagayasu <
> snaga@uptime.jp>
> > > > wrote:
> > > > >>
> > > > >> Hello MADlib community,
> > > > >>
> > > > >> We have created a MADlib 1.10 RC-1, with the artifacts below up
> for
> > a
> > > > vote.
> > > > >>
> > > > >> This will be the 4th release for Apache MADlib (incubating).
> > > > >>
> > > > >> The main goals of this release are:
> > > > >> * new modules (single source shortest path for graph analytics,
> > encode
> > > > >> categorical variables, K-nearest neighbors)
> > > > >> * improvements to existing modules (add grouping support to
> elastic
> > > > >> net and PCA, add cross validation to elastic net, array input for
> > > > >> K-means, verbose output option for DT and RF, limit itemset size
> in
> > > > >> association rules, various madpack installer improvements)
> > > > >> * platform updates (PostgreSQL 9.6)
> > > > >> * bug fixes
> > > > >> * doc improvements
> > > > >>
> > > > >> For more information including release notes, please see:
> > > > >> https://cwiki.apache.org/confluence/display/MADLIB/MADlib+1.10
> > > > >>
> > > > >> *** Please download, review and vote by Sat Feb 18, 2017 @ 6pm PST
> > ***
> > > > >>
> > > > >> We're voting upon the source (tag): rc/1.10.0-rc1
> > > > >> https://github.com/apache/incubator-madlib/tree/rc/1.10.0-rc1
> > > > >>
> > > > >> Source Files:
> > > > >> https://dist.apache.org/repos/dist/dev/incubator/madlib/1.
> > > > >> 10.0-incubating-rc1/
> > > > >>
> > > > >> Commit to be voted upon:
> > > > >> https://github.com/apache/incubator-madlib/commit/
> > > > >> ea17530bfe22a1fde173d7fa83508cbcd9924c20
> > > > >>
> > > > >> KEYS file containing PGP Keys we use to sign the release:
> > > > >> https://dist.apache.org/repos/dist/dev/incubator/madlib/KEYS
> > > > >>
> > > > >> To help in tallying the vote, can PMC members please be sure to
> > > > >> indicate "(binding)" with their vote.
> > > > >>
> > > > >> [ ] +1 approve
> > > > >> [ ] +0 no opinion
> > > > >> [ ] -1 disapprove (and reason why)
> > > > >>
> > > > >> --
> > > > >> Satoshi Nagayasu <sn...@uptime.jp>
> > > > >>
> > > >
> > >
> >
> >
> >
> > --
> > *Ed Espino*
> > *espino@apache.org <es...@apache.org>*
> >
>
--
*Ed Espino*
*espino@apache.org <es...@apache.org>*
Re: [VOTE] MADlib v1.10-rc1
Posted by Ed Espino <es...@apache.org>.
OK (Orhan Kislal),
Thanks for the info (Apache ID signature preferred). That will come in
handy for one of our future Apache HAWQ incubating releases.
Cheers,
-=e
On Thu, Feb 16, 2017 at 12:06 PM, Orhan Kislal <ok...@pivotal.io> wrote:
> Hi Ed,
>
> Thanks for the review. One of the comments from the previous release was a
> preference towards a signature with an Apache id. Since Satoshi-san is not
> an Apache committer yet, I took care of the signing process.
>
> Thanks,
>
> Orhan Kislal
>
> On Thu, Feb 16, 2017 at 11:58 AM, Ed Espino <es...@apache.org> wrote:
>
> > A few MADlib v1.10-rc1 observations from a HAWQ incubator committer.
> >
> > - The Copyright year (2016) in the NOTICE file needs to be updated to
> > 2017. I believe this can be handled in next release.
> > - As it still applies, similar to a past comment by Roman ([VOTE]
> MADlib
> > v1.9.1-rc2
> > <https://lists.apache.org/thread.html/981b4c24eaa2ab069b8e18f7aa4bdd
> > c7a78d3a9dc26bf659af94fcfe@%3Cgeneral.incubator.apache.org%3E>)
> > - *"* name of the top level folder in the archive is weird. The usual
> > practice is to call the top level folder as <project name>-<version
> > ID>*"*
> > (example: *apache-madlib-src-1.10-incubating* instead of
> > *incubator-madlib*)
> > - I'm more curious than anything. Why did Orhan sign the release? I
> was
> > expecting the release manager (Satoshi Nagayasu) to have signed the
> > release.
> > - Checksums and PGP signature are good.
> > - ASF headers check: I spot checked files added (git whatchanged
> > --diff-filter=A) since the last release. ASF headers look good. Nice
> > Job!
> >
> > I was going to try and build but I ran past my allotted time limit for
> this
> > review. Hopefully, I can try this soon.
> >
> > Regards,
> > -=ed espino
> >
> > On Thu, Feb 16, 2017 at 10:05 AM, Orhan Kislal <ok...@pivotal.io>
> wrote:
> >
> > > +1
> > >
> > > Orhan Kislal
> > >
> > > On Thu, Feb 16, 2017 at 9:23 AM, Joe Hellerstein <
> > hellerstein@berkeley.edu
> > > >
> > > wrote:
> > >
> > > > +1
> > > >
> > > > Sent from a telephone.
> > > >
> > > > > On Feb 16, 2017, at 9:17 AM, Frank McQuillan <
> fmcquillan@pivotal.io>
> > > > wrote:
> > > > >
> > > > > +1
> > > > >
> > > > > Frank McQuillan
> > > > >
> > > > >> On Wed, Feb 15, 2017 at 7:27 PM, Satoshi Nagayasu <
> snaga@uptime.jp>
> > > > wrote:
> > > > >>
> > > > >> Hello MADlib community,
> > > > >>
> > > > >> We have created a MADlib 1.10 RC-1, with the artifacts below up
> for
> > a
> > > > vote.
> > > > >>
> > > > >> This will be the 4th release for Apache MADlib (incubating).
> > > > >>
> > > > >> The main goals of this release are:
> > > > >> * new modules (single source shortest path for graph analytics,
> > encode
> > > > >> categorical variables, K-nearest neighbors)
> > > > >> * improvements to existing modules (add grouping support to
> elastic
> > > > >> net and PCA, add cross validation to elastic net, array input for
> > > > >> K-means, verbose output option for DT and RF, limit itemset size
> in
> > > > >> association rules, various madpack installer improvements)
> > > > >> * platform updates (PostgreSQL 9.6)
> > > > >> * bug fixes
> > > > >> * doc improvements
> > > > >>
> > > > >> For more information including release notes, please see:
> > > > >> https://cwiki.apache.org/confluence/display/MADLIB/MADlib+1.10
> > > > >>
> > > > >> *** Please download, review and vote by Sat Feb 18, 2017 @ 6pm PST
> > ***
> > > > >>
> > > > >> We're voting upon the source (tag): rc/1.10.0-rc1
> > > > >> https://github.com/apache/incubator-madlib/tree/rc/1.10.0-rc1
> > > > >>
> > > > >> Source Files:
> > > > >> https://dist.apache.org/repos/dist/dev/incubator/madlib/1.
> > > > >> 10.0-incubating-rc1/
> > > > >>
> > > > >> Commit to be voted upon:
> > > > >> https://github.com/apache/incubator-madlib/commit/
> > > > >> ea17530bfe22a1fde173d7fa83508cbcd9924c20
> > > > >>
> > > > >> KEYS file containing PGP Keys we use to sign the release:
> > > > >> https://dist.apache.org/repos/dist/dev/incubator/madlib/KEYS
> > > > >>
> > > > >> To help in tallying the vote, can PMC members please be sure to
> > > > >> indicate "(binding)" with their vote.
> > > > >>
> > > > >> [ ] +1 approve
> > > > >> [ ] +0 no opinion
> > > > >> [ ] -1 disapprove (and reason why)
> > > > >>
> > > > >> --
> > > > >> Satoshi Nagayasu <sn...@uptime.jp>
> > > > >>
> > > >
> > >
> >
> >
> >
> > --
> > *Ed Espino*
> > *espino@apache.org <es...@apache.org>*
> >
>
--
*Ed Espino*
*espino@apache.org <es...@apache.org>*
Re: [VOTE] MADlib v1.10-rc1
Posted by Nandish Jayaram <nj...@pivotal.io>.
+1
NJ
Sent from my iPhone
> On Feb 16, 2017, at 12:06 PM, Orhan Kislal <ok...@pivotal.io> wrote:
>
> Hi Ed,
>
> Thanks for the review. One of the comments from the previous release was a preference towards a signature with an Apache id. Since Satoshi-san is not an Apache committer yet, I took care of the signing process.
>
> Thanks,
>
> Orhan Kislal
>
>> On Thu, Feb 16, 2017 at 11:58 AM, Ed Espino <es...@apache.org> wrote:
>> A few MADlib v1.10-rc1 observations from a HAWQ incubator committer.
>>
>> - The Copyright year (2016) in the NOTICE file needs to be updated to
>> 2017. I believe this can be handled in next release.
>> - As it still applies, similar to a past comment by Roman ([VOTE] MADlib
>> v1.9.1-rc2
>> <https://lists.apache.org/thread.html/981b4c24eaa2ab069b8e18f7aa4bddc7a78d3a9dc26bf659af94fcfe@%3Cgeneral.incubator.apache.org%3E>)
>> - *"* name of the top level folder in the archive is weird. The usual
>> practice is to call the top level folder as <project name>-<version ID>*"*
>> (example: *apache-madlib-src-1.10-incubating* instead of
>> *incubator-madlib*)
>> - I'm more curious than anything. Why did Orhan sign the release? I was
>> expecting the release manager (Satoshi Nagayasu) to have signed the release.
>> - Checksums and PGP signature are good.
>> - ASF headers check: I spot checked files added (git whatchanged
>> --diff-filter=A) since the last release. ASF headers look good. Nice Job!
>>
>> I was going to try and build but I ran past my allotted time limit for this
>> review. Hopefully, I can try this soon.
>>
>> Regards,
>> -=ed espino
>>
>> On Thu, Feb 16, 2017 at 10:05 AM, Orhan Kislal <ok...@pivotal.io> wrote:
>>
>> > +1
>> >
>> > Orhan Kislal
>> >
>> > On Thu, Feb 16, 2017 at 9:23 AM, Joe Hellerstein <hellerstein@berkeley.edu
>> > >
>> > wrote:
>> >
>> > > +1
>> > >
>> > > Sent from a telephone.
>> > >
>> > > > On Feb 16, 2017, at 9:17 AM, Frank McQuillan <fm...@pivotal.io>
>> > > wrote:
>> > > >
>> > > > +1
>> > > >
>> > > > Frank McQuillan
>> > > >
>> > > >> On Wed, Feb 15, 2017 at 7:27 PM, Satoshi Nagayasu <sn...@uptime.jp>
>> > > wrote:
>> > > >>
>> > > >> Hello MADlib community,
>> > > >>
>> > > >> We have created a MADlib 1.10 RC-1, with the artifacts below up for a
>> > > vote.
>> > > >>
>> > > >> This will be the 4th release for Apache MADlib (incubating).
>> > > >>
>> > > >> The main goals of this release are:
>> > > >> * new modules (single source shortest path for graph analytics, encode
>> > > >> categorical variables, K-nearest neighbors)
>> > > >> * improvements to existing modules (add grouping support to elastic
>> > > >> net and PCA, add cross validation to elastic net, array input for
>> > > >> K-means, verbose output option for DT and RF, limit itemset size in
>> > > >> association rules, various madpack installer improvements)
>> > > >> * platform updates (PostgreSQL 9.6)
>> > > >> * bug fixes
>> > > >> * doc improvements
>> > > >>
>> > > >> For more information including release notes, please see:
>> > > >> https://cwiki.apache.org/confluence/display/MADLIB/MADlib+1.10
>> > > >>
>> > > >> *** Please download, review and vote by Sat Feb 18, 2017 @ 6pm PST ***
>> > > >>
>> > > >> We're voting upon the source (tag): rc/1.10.0-rc1
>> > > >> https://github.com/apache/incubator-madlib/tree/rc/1.10.0-rc1
>> > > >>
>> > > >> Source Files:
>> > > >> https://dist.apache.org/repos/dist/dev/incubator/madlib/1.
>> > > >> 10.0-incubating-rc1/
>> > > >>
>> > > >> Commit to be voted upon:
>> > > >> https://github.com/apache/incubator-madlib/commit/
>> > > >> ea17530bfe22a1fde173d7fa83508cbcd9924c20
>> > > >>
>> > > >> KEYS file containing PGP Keys we use to sign the release:
>> > > >> https://dist.apache.org/repos/dist/dev/incubator/madlib/KEYS
>> > > >>
>> > > >> To help in tallying the vote, can PMC members please be sure to
>> > > >> indicate "(binding)" with their vote.
>> > > >>
>> > > >> [ ] +1 approve
>> > > >> [ ] +0 no opinion
>> > > >> [ ] -1 disapprove (and reason why)
>> > > >>
>> > > >> --
>> > > >> Satoshi Nagayasu <sn...@uptime.jp>
>> > > >>
>> > >
>> >
>>
>>
>>
>> --
>> *Ed Espino*
>> *espino@apache.org <es...@apache.org>*
>
Re: [VOTE] MADlib v1.10-rc1
Posted by Ed Espino <es...@apache.org>.
Orhan,
One more mildly interesting thing I noticed, I can't use the gsha512sum and
gmd5sum commands to validate the corresponding checksum values easily. Even
though the commands (see below links) available on the Apache site do
generate correct checksums, they aren't conducive to a quick validation of
them.
- What Is An MD5 Checksum?
<https://www.apache.org/dev/release-signing.html#md5>
- What is a SHA checksum?
<https://www.apache.org/dev/release-signing.html#sha-checksum>
For reference, here are the processes and validation steps we use for the
Apache HAWQ incubator project. They may be of some use to your project and
help those validating checksums.
- Create the Release Candidate
<https://cwiki.apache.org/confluence/display/HAWQ/Release+Process%3A+Step+by+step+guide#ReleaseProcess:Stepbystepguide-CreatetheReleaseCandidate>
Hope this helps,
-=e
On Thu, Feb 16, 2017 at 12:06 PM, Orhan Kislal <ok...@pivotal.io> wrote:
> Hi Ed,
>
> Thanks for the review. One of the comments from the previous release was a
> preference towards a signature with an Apache id. Since Satoshi-san is not
> an Apache committer yet, I took care of the signing process.
>
> Thanks,
>
> Orhan Kislal
>
> On Thu, Feb 16, 2017 at 11:58 AM, Ed Espino <es...@apache.org> wrote:
>
> > A few MADlib v1.10-rc1 observations from a HAWQ incubator committer.
> >
> > - The Copyright year (2016) in the NOTICE file needs to be updated to
> > 2017. I believe this can be handled in next release.
> > - As it still applies, similar to a past comment by Roman ([VOTE]
> MADlib
> > v1.9.1-rc2
> > <https://lists.apache.org/thread.html/981b4c24eaa2ab069b8e18f7aa4bdd
> > c7a78d3a9dc26bf659af94fcfe@%3Cgeneral.incubator.apache.org%3E>)
> > - *"* name of the top level folder in the archive is weird. The usual
> > practice is to call the top level folder as <project name>-<version
> > ID>*"*
> > (example: *apache-madlib-src-1.10-incubating* instead of
> > *incubator-madlib*)
> > - I'm more curious than anything. Why did Orhan sign the release? I
> was
> > expecting the release manager (Satoshi Nagayasu) to have signed the
> > release.
> > - Checksums and PGP signature are good.
> > - ASF headers check: I spot checked files added (git whatchanged
> > --diff-filter=A) since the last release. ASF headers look good. Nice
> > Job!
> >
> > I was going to try and build but I ran past my allotted time limit for
> this
> > review. Hopefully, I can try this soon.
> >
> > Regards,
> > -=ed espino
> >
> > On Thu, Feb 16, 2017 at 10:05 AM, Orhan Kislal <ok...@pivotal.io>
> wrote:
> >
> > > +1
> > >
> > > Orhan Kislal
> > >
> > > On Thu, Feb 16, 2017 at 9:23 AM, Joe Hellerstein <
> > hellerstein@berkeley.edu
> > > >
> > > wrote:
> > >
> > > > +1
> > > >
> > > > Sent from a telephone.
> > > >
> > > > > On Feb 16, 2017, at 9:17 AM, Frank McQuillan <
> fmcquillan@pivotal.io>
> > > > wrote:
> > > > >
> > > > > +1
> > > > >
> > > > > Frank McQuillan
> > > > >
> > > > >> On Wed, Feb 15, 2017 at 7:27 PM, Satoshi Nagayasu <
> snaga@uptime.jp>
> > > > wrote:
> > > > >>
> > > > >> Hello MADlib community,
> > > > >>
> > > > >> We have created a MADlib 1.10 RC-1, with the artifacts below up
> for
> > a
> > > > vote.
> > > > >>
> > > > >> This will be the 4th release for Apache MADlib (incubating).
> > > > >>
> > > > >> The main goals of this release are:
> > > > >> * new modules (single source shortest path for graph analytics,
> > encode
> > > > >> categorical variables, K-nearest neighbors)
> > > > >> * improvements to existing modules (add grouping support to
> elastic
> > > > >> net and PCA, add cross validation to elastic net, array input for
> > > > >> K-means, verbose output option for DT and RF, limit itemset size
> in
> > > > >> association rules, various madpack installer improvements)
> > > > >> * platform updates (PostgreSQL 9.6)
> > > > >> * bug fixes
> > > > >> * doc improvements
> > > > >>
> > > > >> For more information including release notes, please see:
> > > > >> https://cwiki.apache.org/confluence/display/MADLIB/MADlib+1.10
> > > > >>
> > > > >> *** Please download, review and vote by Sat Feb 18, 2017 @ 6pm PST
> > ***
> > > > >>
> > > > >> We're voting upon the source (tag): rc/1.10.0-rc1
> > > > >> https://github.com/apache/incubator-madlib/tree/rc/1.10.0-rc1
> > > > >>
> > > > >> Source Files:
> > > > >> https://dist.apache.org/repos/dist/dev/incubator/madlib/1.
> > > > >> 10.0-incubating-rc1/
> > > > >>
> > > > >> Commit to be voted upon:
> > > > >> https://github.com/apache/incubator-madlib/commit/
> > > > >> ea17530bfe22a1fde173d7fa83508cbcd9924c20
> > > > >>
> > > > >> KEYS file containing PGP Keys we use to sign the release:
> > > > >> https://dist.apache.org/repos/dist/dev/incubator/madlib/KEYS
> > > > >>
> > > > >> To help in tallying the vote, can PMC members please be sure to
> > > > >> indicate "(binding)" with their vote.
> > > > >>
> > > > >> [ ] +1 approve
> > > > >> [ ] +0 no opinion
> > > > >> [ ] -1 disapprove (and reason why)
> > > > >>
> > > > >> --
> > > > >> Satoshi Nagayasu <sn...@uptime.jp>
> > > > >>
> > > >
> > >
> >
> >
> >
> > --
> > *Ed Espino*
> > *espino@apache.org <es...@apache.org>*
> >
>
--
*Ed Espino*
*espino@apache.org <es...@apache.org>*
Re: [VOTE] MADlib v1.10-rc1
Posted by Ed Espino <es...@apache.org>.
OK (Orhan Kislal),
Thanks for the info (Apache ID signature preferred). That will come in
handy for one of our future Apache HAWQ incubating releases.
Cheers,
-=e
On Thu, Feb 16, 2017 at 12:06 PM, Orhan Kislal <ok...@pivotal.io> wrote:
> Hi Ed,
>
> Thanks for the review. One of the comments from the previous release was a
> preference towards a signature with an Apache id. Since Satoshi-san is not
> an Apache committer yet, I took care of the signing process.
>
> Thanks,
>
> Orhan Kislal
>
> On Thu, Feb 16, 2017 at 11:58 AM, Ed Espino <es...@apache.org> wrote:
>
> > A few MADlib v1.10-rc1 observations from a HAWQ incubator committer.
> >
> > - The Copyright year (2016) in the NOTICE file needs to be updated to
> > 2017. I believe this can be handled in next release.
> > - As it still applies, similar to a past comment by Roman ([VOTE]
> MADlib
> > v1.9.1-rc2
> > <https://lists.apache.org/thread.html/981b4c24eaa2ab069b8e18f7aa4bdd
> > c7a78d3a9dc26bf659af94fcfe@%3Cgeneral.incubator.apache.org%3E>)
> > - *"* name of the top level folder in the archive is weird. The usual
> > practice is to call the top level folder as <project name>-<version
> > ID>*"*
> > (example: *apache-madlib-src-1.10-incubating* instead of
> > *incubator-madlib*)
> > - I'm more curious than anything. Why did Orhan sign the release? I
> was
> > expecting the release manager (Satoshi Nagayasu) to have signed the
> > release.
> > - Checksums and PGP signature are good.
> > - ASF headers check: I spot checked files added (git whatchanged
> > --diff-filter=A) since the last release. ASF headers look good. Nice
> > Job!
> >
> > I was going to try and build but I ran past my allotted time limit for
> this
> > review. Hopefully, I can try this soon.
> >
> > Regards,
> > -=ed espino
> >
> > On Thu, Feb 16, 2017 at 10:05 AM, Orhan Kislal <ok...@pivotal.io>
> wrote:
> >
> > > +1
> > >
> > > Orhan Kislal
> > >
> > > On Thu, Feb 16, 2017 at 9:23 AM, Joe Hellerstein <
> > hellerstein@berkeley.edu
> > > >
> > > wrote:
> > >
> > > > +1
> > > >
> > > > Sent from a telephone.
> > > >
> > > > > On Feb 16, 2017, at 9:17 AM, Frank McQuillan <
> fmcquillan@pivotal.io>
> > > > wrote:
> > > > >
> > > > > +1
> > > > >
> > > > > Frank McQuillan
> > > > >
> > > > >> On Wed, Feb 15, 2017 at 7:27 PM, Satoshi Nagayasu <
> snaga@uptime.jp>
> > > > wrote:
> > > > >>
> > > > >> Hello MADlib community,
> > > > >>
> > > > >> We have created a MADlib 1.10 RC-1, with the artifacts below up
> for
> > a
> > > > vote.
> > > > >>
> > > > >> This will be the 4th release for Apache MADlib (incubating).
> > > > >>
> > > > >> The main goals of this release are:
> > > > >> * new modules (single source shortest path for graph analytics,
> > encode
> > > > >> categorical variables, K-nearest neighbors)
> > > > >> * improvements to existing modules (add grouping support to
> elastic
> > > > >> net and PCA, add cross validation to elastic net, array input for
> > > > >> K-means, verbose output option for DT and RF, limit itemset size
> in
> > > > >> association rules, various madpack installer improvements)
> > > > >> * platform updates (PostgreSQL 9.6)
> > > > >> * bug fixes
> > > > >> * doc improvements
> > > > >>
> > > > >> For more information including release notes, please see:
> > > > >> https://cwiki.apache.org/confluence/display/MADLIB/MADlib+1.10
> > > > >>
> > > > >> *** Please download, review and vote by Sat Feb 18, 2017 @ 6pm PST
> > ***
> > > > >>
> > > > >> We're voting upon the source (tag): rc/1.10.0-rc1
> > > > >> https://github.com/apache/incubator-madlib/tree/rc/1.10.0-rc1
> > > > >>
> > > > >> Source Files:
> > > > >> https://dist.apache.org/repos/dist/dev/incubator/madlib/1.
> > > > >> 10.0-incubating-rc1/
> > > > >>
> > > > >> Commit to be voted upon:
> > > > >> https://github.com/apache/incubator-madlib/commit/
> > > > >> ea17530bfe22a1fde173d7fa83508cbcd9924c20
> > > > >>
> > > > >> KEYS file containing PGP Keys we use to sign the release:
> > > > >> https://dist.apache.org/repos/dist/dev/incubator/madlib/KEYS
> > > > >>
> > > > >> To help in tallying the vote, can PMC members please be sure to
> > > > >> indicate "(binding)" with their vote.
> > > > >>
> > > > >> [ ] +1 approve
> > > > >> [ ] +0 no opinion
> > > > >> [ ] -1 disapprove (and reason why)
> > > > >>
> > > > >> --
> > > > >> Satoshi Nagayasu <sn...@uptime.jp>
> > > > >>
> > > >
> > >
> >
> >
> >
> > --
> > *Ed Espino*
> > *espino@apache.org <es...@apache.org>*
> >
>
--
*Ed Espino*
*espino@apache.org <es...@apache.org>*
Re: [VOTE] MADlib v1.10-rc1
Posted by Orhan Kislal <ok...@pivotal.io>.
Hi Ed,
Thanks for the review. One of the comments from the previous release was a
preference towards a signature with an Apache id. Since Satoshi-san is not
an Apache committer yet, I took care of the signing process.
Thanks,
Orhan Kislal
On Thu, Feb 16, 2017 at 11:58 AM, Ed Espino <es...@apache.org> wrote:
> A few MADlib v1.10-rc1 observations from a HAWQ incubator committer.
>
> - The Copyright year (2016) in the NOTICE file needs to be updated to
> 2017. I believe this can be handled in next release.
> - As it still applies, similar to a past comment by Roman ([VOTE] MADlib
> v1.9.1-rc2
> <https://lists.apache.org/thread.html/981b4c24eaa2ab069b8e18f7aa4bdd
> c7a78d3a9dc26bf659af94fcfe@%3Cgeneral.incubator.apache.org%3E>)
> - *"* name of the top level folder in the archive is weird. The usual
> practice is to call the top level folder as <project name>-<version
> ID>*"*
> (example: *apache-madlib-src-1.10-incubating* instead of
> *incubator-madlib*)
> - I'm more curious than anything. Why did Orhan sign the release? I was
> expecting the release manager (Satoshi Nagayasu) to have signed the
> release.
> - Checksums and PGP signature are good.
> - ASF headers check: I spot checked files added (git whatchanged
> --diff-filter=A) since the last release. ASF headers look good. Nice
> Job!
>
> I was going to try and build but I ran past my allotted time limit for this
> review. Hopefully, I can try this soon.
>
> Regards,
> -=ed espino
>
> On Thu, Feb 16, 2017 at 10:05 AM, Orhan Kislal <ok...@pivotal.io> wrote:
>
> > +1
> >
> > Orhan Kislal
> >
> > On Thu, Feb 16, 2017 at 9:23 AM, Joe Hellerstein <
> hellerstein@berkeley.edu
> > >
> > wrote:
> >
> > > +1
> > >
> > > Sent from a telephone.
> > >
> > > > On Feb 16, 2017, at 9:17 AM, Frank McQuillan <fm...@pivotal.io>
> > > wrote:
> > > >
> > > > +1
> > > >
> > > > Frank McQuillan
> > > >
> > > >> On Wed, Feb 15, 2017 at 7:27 PM, Satoshi Nagayasu <sn...@uptime.jp>
> > > wrote:
> > > >>
> > > >> Hello MADlib community,
> > > >>
> > > >> We have created a MADlib 1.10 RC-1, with the artifacts below up for
> a
> > > vote.
> > > >>
> > > >> This will be the 4th release for Apache MADlib (incubating).
> > > >>
> > > >> The main goals of this release are:
> > > >> * new modules (single source shortest path for graph analytics,
> encode
> > > >> categorical variables, K-nearest neighbors)
> > > >> * improvements to existing modules (add grouping support to elastic
> > > >> net and PCA, add cross validation to elastic net, array input for
> > > >> K-means, verbose output option for DT and RF, limit itemset size in
> > > >> association rules, various madpack installer improvements)
> > > >> * platform updates (PostgreSQL 9.6)
> > > >> * bug fixes
> > > >> * doc improvements
> > > >>
> > > >> For more information including release notes, please see:
> > > >> https://cwiki.apache.org/confluence/display/MADLIB/MADlib+1.10
> > > >>
> > > >> *** Please download, review and vote by Sat Feb 18, 2017 @ 6pm PST
> ***
> > > >>
> > > >> We're voting upon the source (tag): rc/1.10.0-rc1
> > > >> https://github.com/apache/incubator-madlib/tree/rc/1.10.0-rc1
> > > >>
> > > >> Source Files:
> > > >> https://dist.apache.org/repos/dist/dev/incubator/madlib/1.
> > > >> 10.0-incubating-rc1/
> > > >>
> > > >> Commit to be voted upon:
> > > >> https://github.com/apache/incubator-madlib/commit/
> > > >> ea17530bfe22a1fde173d7fa83508cbcd9924c20
> > > >>
> > > >> KEYS file containing PGP Keys we use to sign the release:
> > > >> https://dist.apache.org/repos/dist/dev/incubator/madlib/KEYS
> > > >>
> > > >> To help in tallying the vote, can PMC members please be sure to
> > > >> indicate "(binding)" with their vote.
> > > >>
> > > >> [ ] +1 approve
> > > >> [ ] +0 no opinion
> > > >> [ ] -1 disapprove (and reason why)
> > > >>
> > > >> --
> > > >> Satoshi Nagayasu <sn...@uptime.jp>
> > > >>
> > >
> >
>
>
>
> --
> *Ed Espino*
> *espino@apache.org <es...@apache.org>*
>
Re: [VOTE] MADlib v1.10-rc1
Posted by Satoshi Nagayasu <sn...@uptime.jp>.
Hi,
2017-02-17 4:58 GMT+09:00 Ed Espino <es...@apache.org>:
> A few MADlib v1.10-rc1 observations from a HAWQ incubator committer.
>
> - The Copyright year (2016) in the NOTICE file needs to be updated to
> 2017. I believe this can be handled in next release.
I have created a pull request to fix this.
https://github.com/apache/incubator-madlib/pull/104
> - As it still applies, similar to a past comment by Roman ([VOTE] MADlib
> v1.9.1-rc2
> <https://lists.apache.org/thread.html/981b4c24eaa2ab069b8e18f7aa4bddc7a78d3a9dc26bf659af94fcfe@%3Cgeneral.incubator.apache.org%3E>)
Here is the checklist from the previous release process:
> * imported keys from the provided KEYS file
> * checksums and signatures
> * the release archive is identical to the content of the tag
> * DICLAIMER, LICENSE and NOTICE are all as they should be
Hope these would go well. :D
> * ran RAT check (via mvn verify) and made sure that the license
> headers in files get properly recognized and the excluded BSD
> licensed files are correctly excluded
> * all brand new files in this release have ALv2 headers
Yes, you all can check the RAT result here (see rat.txt):
http://161.202.86.165/build/incubator-madlib/
> Minor nits that I stills strongly suggest you address in future release:
> * name of the top level folder in the archive is weird. The usual practice
> is to call the top level folder as <project name>-<version ID>
I would like to fix this as following.
> - *"* name of the top level folder in the archive is weird. The usual
> practice is to call the top level folder as <project name>-<version ID>*"*
> (example: *apache-madlib-src-1.10-incubating* instead of
> *incubator-madlib*)
I would like to suggest the name of the top directory "incubator-madlib-1.10.0".
My artifacts for testing purpose are available here:
http://161.202.86.165/build/incubator-madlib/
These artifacts are generated automatically in every 3 hours with running
install-check (against PG 9.6) and Apace RAT.
> - I'm more curious than anything. Why did Orhan sign the release? I was
> expecting the release manager (Satoshi Nagayasu) to have signed the release.
Right, I need Orhan's help as he mentioned in another reply.
> - Checksums and PGP signature are good.
> - ASF headers check: I spot checked files added (git whatchanged
> --diff-filter=A) since the last release. ASF headers look good. Nice Job!
Hope these could be ok. :D
Regards,
--
Satoshi Nagayasu <sn...@uptime.jp>
Re: [VOTE] MADlib v1.10-rc1
Posted by Satoshi Nagayasu <sn...@uptime.jp>.
Hi,
2017-02-17 4:58 GMT+09:00 Ed Espino <es...@apache.org>:
> A few MADlib v1.10-rc1 observations from a HAWQ incubator committer.
>
> - The Copyright year (2016) in the NOTICE file needs to be updated to
> 2017. I believe this can be handled in next release.
I have created a pull request to fix this.
https://github.com/apache/incubator-madlib/pull/104
> - As it still applies, similar to a past comment by Roman ([VOTE] MADlib
> v1.9.1-rc2
> <https://lists.apache.org/thread.html/981b4c24eaa2ab069b8e18f7aa4bddc7a78d3a9dc26bf659af94fcfe@%3Cgeneral.incubator.apache.org%3E>)
Here is the checklist from the previous release process:
> * imported keys from the provided KEYS file
> * checksums and signatures
> * the release archive is identical to the content of the tag
> * DICLAIMER, LICENSE and NOTICE are all as they should be
Hope these would go well. :D
> * ran RAT check (via mvn verify) and made sure that the license
> headers in files get properly recognized and the excluded BSD
> licensed files are correctly excluded
> * all brand new files in this release have ALv2 headers
Yes, you all can check the RAT result here (see rat.txt):
http://161.202.86.165/build/incubator-madlib/
> Minor nits that I stills strongly suggest you address in future release:
> * name of the top level folder in the archive is weird. The usual practice
> is to call the top level folder as <project name>-<version ID>
I would like to fix this as following.
> - *"* name of the top level folder in the archive is weird. The usual
> practice is to call the top level folder as <project name>-<version ID>*"*
> (example: *apache-madlib-src-1.10-incubating* instead of
> *incubator-madlib*)
I would like to suggest the name of the top directory "incubator-madlib-1.10.0".
My artifacts for testing purpose are available here:
http://161.202.86.165/build/incubator-madlib/
These artifacts are generated automatically in every 3 hours with running
install-check (against PG 9.6) and Apace RAT.
> - I'm more curious than anything. Why did Orhan sign the release? I was
> expecting the release manager (Satoshi Nagayasu) to have signed the release.
Right, I need Orhan's help as he mentioned in another reply.
> - Checksums and PGP signature are good.
> - ASF headers check: I spot checked files added (git whatchanged
> --diff-filter=A) since the last release. ASF headers look good. Nice Job!
Hope these could be ok. :D
Regards,
--
Satoshi Nagayasu <sn...@uptime.jp>
Re: [VOTE] MADlib v1.10-rc1
Posted by Orhan Kislal <ok...@pivotal.io>.
Hi Ed,
Thanks for the review. One of the comments from the previous release was a
preference towards a signature with an Apache id. Since Satoshi-san is not
an Apache committer yet, I took care of the signing process.
Thanks,
Orhan Kislal
On Thu, Feb 16, 2017 at 11:58 AM, Ed Espino <es...@apache.org> wrote:
> A few MADlib v1.10-rc1 observations from a HAWQ incubator committer.
>
> - The Copyright year (2016) in the NOTICE file needs to be updated to
> 2017. I believe this can be handled in next release.
> - As it still applies, similar to a past comment by Roman ([VOTE] MADlib
> v1.9.1-rc2
> <https://lists.apache.org/thread.html/981b4c24eaa2ab069b8e18f7aa4bdd
> c7a78d3a9dc26bf659af94fcfe@%3Cgeneral.incubator.apache.org%3E>)
> - *"* name of the top level folder in the archive is weird. The usual
> practice is to call the top level folder as <project name>-<version
> ID>*"*
> (example: *apache-madlib-src-1.10-incubating* instead of
> *incubator-madlib*)
> - I'm more curious than anything. Why did Orhan sign the release? I was
> expecting the release manager (Satoshi Nagayasu) to have signed the
> release.
> - Checksums and PGP signature are good.
> - ASF headers check: I spot checked files added (git whatchanged
> --diff-filter=A) since the last release. ASF headers look good. Nice
> Job!
>
> I was going to try and build but I ran past my allotted time limit for this
> review. Hopefully, I can try this soon.
>
> Regards,
> -=ed espino
>
> On Thu, Feb 16, 2017 at 10:05 AM, Orhan Kislal <ok...@pivotal.io> wrote:
>
> > +1
> >
> > Orhan Kislal
> >
> > On Thu, Feb 16, 2017 at 9:23 AM, Joe Hellerstein <
> hellerstein@berkeley.edu
> > >
> > wrote:
> >
> > > +1
> > >
> > > Sent from a telephone.
> > >
> > > > On Feb 16, 2017, at 9:17 AM, Frank McQuillan <fm...@pivotal.io>
> > > wrote:
> > > >
> > > > +1
> > > >
> > > > Frank McQuillan
> > > >
> > > >> On Wed, Feb 15, 2017 at 7:27 PM, Satoshi Nagayasu <sn...@uptime.jp>
> > > wrote:
> > > >>
> > > >> Hello MADlib community,
> > > >>
> > > >> We have created a MADlib 1.10 RC-1, with the artifacts below up for
> a
> > > vote.
> > > >>
> > > >> This will be the 4th release for Apache MADlib (incubating).
> > > >>
> > > >> The main goals of this release are:
> > > >> * new modules (single source shortest path for graph analytics,
> encode
> > > >> categorical variables, K-nearest neighbors)
> > > >> * improvements to existing modules (add grouping support to elastic
> > > >> net and PCA, add cross validation to elastic net, array input for
> > > >> K-means, verbose output option for DT and RF, limit itemset size in
> > > >> association rules, various madpack installer improvements)
> > > >> * platform updates (PostgreSQL 9.6)
> > > >> * bug fixes
> > > >> * doc improvements
> > > >>
> > > >> For more information including release notes, please see:
> > > >> https://cwiki.apache.org/confluence/display/MADLIB/MADlib+1.10
> > > >>
> > > >> *** Please download, review and vote by Sat Feb 18, 2017 @ 6pm PST
> ***
> > > >>
> > > >> We're voting upon the source (tag): rc/1.10.0-rc1
> > > >> https://github.com/apache/incubator-madlib/tree/rc/1.10.0-rc1
> > > >>
> > > >> Source Files:
> > > >> https://dist.apache.org/repos/dist/dev/incubator/madlib/1.
> > > >> 10.0-incubating-rc1/
> > > >>
> > > >> Commit to be voted upon:
> > > >> https://github.com/apache/incubator-madlib/commit/
> > > >> ea17530bfe22a1fde173d7fa83508cbcd9924c20
> > > >>
> > > >> KEYS file containing PGP Keys we use to sign the release:
> > > >> https://dist.apache.org/repos/dist/dev/incubator/madlib/KEYS
> > > >>
> > > >> To help in tallying the vote, can PMC members please be sure to
> > > >> indicate "(binding)" with their vote.
> > > >>
> > > >> [ ] +1 approve
> > > >> [ ] +0 no opinion
> > > >> [ ] -1 disapprove (and reason why)
> > > >>
> > > >> --
> > > >> Satoshi Nagayasu <sn...@uptime.jp>
> > > >>
> > >
> >
>
>
>
> --
> *Ed Espino*
> *espino@apache.org <es...@apache.org>*
>
Re: [VOTE] MADlib v1.10-rc1
Posted by Ed Espino <es...@apache.org>.
A few MADlib v1.10-rc1 observations from a HAWQ incubator committer.
- The Copyright year (2016) in the NOTICE file needs to be updated to
2017. I believe this can be handled in next release.
- As it still applies, similar to a past comment by Roman ([VOTE] MADlib
v1.9.1-rc2
<https://lists.apache.org/thread.html/981b4c24eaa2ab069b8e18f7aa4bddc7a78d3a9dc26bf659af94fcfe@%3Cgeneral.incubator.apache.org%3E>)
- *"* name of the top level folder in the archive is weird. The usual
practice is to call the top level folder as <project name>-<version ID>*"*
(example: *apache-madlib-src-1.10-incubating* instead of
*incubator-madlib*)
- I'm more curious than anything. Why did Orhan sign the release? I was
expecting the release manager (Satoshi Nagayasu) to have signed the release.
- Checksums and PGP signature are good.
- ASF headers check: I spot checked files added (git whatchanged
--diff-filter=A) since the last release. ASF headers look good. Nice Job!
I was going to try and build but I ran past my allotted time limit for this
review. Hopefully, I can try this soon.
Regards,
-=ed espino
On Thu, Feb 16, 2017 at 10:05 AM, Orhan Kislal <ok...@pivotal.io> wrote:
> +1
>
> Orhan Kislal
>
> On Thu, Feb 16, 2017 at 9:23 AM, Joe Hellerstein <hellerstein@berkeley.edu
> >
> wrote:
>
> > +1
> >
> > Sent from a telephone.
> >
> > > On Feb 16, 2017, at 9:17 AM, Frank McQuillan <fm...@pivotal.io>
> > wrote:
> > >
> > > +1
> > >
> > > Frank McQuillan
> > >
> > >> On Wed, Feb 15, 2017 at 7:27 PM, Satoshi Nagayasu <sn...@uptime.jp>
> > wrote:
> > >>
> > >> Hello MADlib community,
> > >>
> > >> We have created a MADlib 1.10 RC-1, with the artifacts below up for a
> > vote.
> > >>
> > >> This will be the 4th release for Apache MADlib (incubating).
> > >>
> > >> The main goals of this release are:
> > >> * new modules (single source shortest path for graph analytics, encode
> > >> categorical variables, K-nearest neighbors)
> > >> * improvements to existing modules (add grouping support to elastic
> > >> net and PCA, add cross validation to elastic net, array input for
> > >> K-means, verbose output option for DT and RF, limit itemset size in
> > >> association rules, various madpack installer improvements)
> > >> * platform updates (PostgreSQL 9.6)
> > >> * bug fixes
> > >> * doc improvements
> > >>
> > >> For more information including release notes, please see:
> > >> https://cwiki.apache.org/confluence/display/MADLIB/MADlib+1.10
> > >>
> > >> *** Please download, review and vote by Sat Feb 18, 2017 @ 6pm PST ***
> > >>
> > >> We're voting upon the source (tag): rc/1.10.0-rc1
> > >> https://github.com/apache/incubator-madlib/tree/rc/1.10.0-rc1
> > >>
> > >> Source Files:
> > >> https://dist.apache.org/repos/dist/dev/incubator/madlib/1.
> > >> 10.0-incubating-rc1/
> > >>
> > >> Commit to be voted upon:
> > >> https://github.com/apache/incubator-madlib/commit/
> > >> ea17530bfe22a1fde173d7fa83508cbcd9924c20
> > >>
> > >> KEYS file containing PGP Keys we use to sign the release:
> > >> https://dist.apache.org/repos/dist/dev/incubator/madlib/KEYS
> > >>
> > >> To help in tallying the vote, can PMC members please be sure to
> > >> indicate "(binding)" with their vote.
> > >>
> > >> [ ] +1 approve
> > >> [ ] +0 no opinion
> > >> [ ] -1 disapprove (and reason why)
> > >>
> > >> --
> > >> Satoshi Nagayasu <sn...@uptime.jp>
> > >>
> >
>
--
*Ed Espino*
*espino@apache.org <es...@apache.org>*
Re: [VOTE] MADlib v1.10-rc1
Posted by Ed Espino <es...@apache.org>.
A few MADlib v1.10-rc1 observations from a HAWQ incubator committer.
- The Copyright year (2016) in the NOTICE file needs to be updated to
2017. I believe this can be handled in next release.
- As it still applies, similar to a past comment by Roman ([VOTE] MADlib
v1.9.1-rc2
<https://lists.apache.org/thread.html/981b4c24eaa2ab069b8e18f7aa4bddc7a78d3a9dc26bf659af94fcfe@%3Cgeneral.incubator.apache.org%3E>)
- *"* name of the top level folder in the archive is weird. The usual
practice is to call the top level folder as <project name>-<version ID>*"*
(example: *apache-madlib-src-1.10-incubating* instead of
*incubator-madlib*)
- I'm more curious than anything. Why did Orhan sign the release? I was
expecting the release manager (Satoshi Nagayasu) to have signed the release.
- Checksums and PGP signature are good.
- ASF headers check: I spot checked files added (git whatchanged
--diff-filter=A) since the last release. ASF headers look good. Nice Job!
I was going to try and build but I ran past my allotted time limit for this
review. Hopefully, I can try this soon.
Regards,
-=ed espino
On Thu, Feb 16, 2017 at 10:05 AM, Orhan Kislal <ok...@pivotal.io> wrote:
> +1
>
> Orhan Kislal
>
> On Thu, Feb 16, 2017 at 9:23 AM, Joe Hellerstein <hellerstein@berkeley.edu
> >
> wrote:
>
> > +1
> >
> > Sent from a telephone.
> >
> > > On Feb 16, 2017, at 9:17 AM, Frank McQuillan <fm...@pivotal.io>
> > wrote:
> > >
> > > +1
> > >
> > > Frank McQuillan
> > >
> > >> On Wed, Feb 15, 2017 at 7:27 PM, Satoshi Nagayasu <sn...@uptime.jp>
> > wrote:
> > >>
> > >> Hello MADlib community,
> > >>
> > >> We have created a MADlib 1.10 RC-1, with the artifacts below up for a
> > vote.
> > >>
> > >> This will be the 4th release for Apache MADlib (incubating).
> > >>
> > >> The main goals of this release are:
> > >> * new modules (single source shortest path for graph analytics, encode
> > >> categorical variables, K-nearest neighbors)
> > >> * improvements to existing modules (add grouping support to elastic
> > >> net and PCA, add cross validation to elastic net, array input for
> > >> K-means, verbose output option for DT and RF, limit itemset size in
> > >> association rules, various madpack installer improvements)
> > >> * platform updates (PostgreSQL 9.6)
> > >> * bug fixes
> > >> * doc improvements
> > >>
> > >> For more information including release notes, please see:
> > >> https://cwiki.apache.org/confluence/display/MADLIB/MADlib+1.10
> > >>
> > >> *** Please download, review and vote by Sat Feb 18, 2017 @ 6pm PST ***
> > >>
> > >> We're voting upon the source (tag): rc/1.10.0-rc1
> > >> https://github.com/apache/incubator-madlib/tree/rc/1.10.0-rc1
> > >>
> > >> Source Files:
> > >> https://dist.apache.org/repos/dist/dev/incubator/madlib/1.
> > >> 10.0-incubating-rc1/
> > >>
> > >> Commit to be voted upon:
> > >> https://github.com/apache/incubator-madlib/commit/
> > >> ea17530bfe22a1fde173d7fa83508cbcd9924c20
> > >>
> > >> KEYS file containing PGP Keys we use to sign the release:
> > >> https://dist.apache.org/repos/dist/dev/incubator/madlib/KEYS
> > >>
> > >> To help in tallying the vote, can PMC members please be sure to
> > >> indicate "(binding)" with their vote.
> > >>
> > >> [ ] +1 approve
> > >> [ ] +0 no opinion
> > >> [ ] -1 disapprove (and reason why)
> > >>
> > >> --
> > >> Satoshi Nagayasu <sn...@uptime.jp>
> > >>
> >
>
--
*Ed Espino*
*espino@apache.org <es...@apache.org>*
Re: [VOTE] MADlib v1.10-rc1
Posted by Orhan Kislal <ok...@pivotal.io>.
+1
Orhan Kislal
On Thu, Feb 16, 2017 at 9:23 AM, Joe Hellerstein <he...@berkeley.edu>
wrote:
> +1
>
> Sent from a telephone.
>
> > On Feb 16, 2017, at 9:17 AM, Frank McQuillan <fm...@pivotal.io>
> wrote:
> >
> > +1
> >
> > Frank McQuillan
> >
> >> On Wed, Feb 15, 2017 at 7:27 PM, Satoshi Nagayasu <sn...@uptime.jp>
> wrote:
> >>
> >> Hello MADlib community,
> >>
> >> We have created a MADlib 1.10 RC-1, with the artifacts below up for a
> vote.
> >>
> >> This will be the 4th release for Apache MADlib (incubating).
> >>
> >> The main goals of this release are:
> >> * new modules (single source shortest path for graph analytics, encode
> >> categorical variables, K-nearest neighbors)
> >> * improvements to existing modules (add grouping support to elastic
> >> net and PCA, add cross validation to elastic net, array input for
> >> K-means, verbose output option for DT and RF, limit itemset size in
> >> association rules, various madpack installer improvements)
> >> * platform updates (PostgreSQL 9.6)
> >> * bug fixes
> >> * doc improvements
> >>
> >> For more information including release notes, please see:
> >> https://cwiki.apache.org/confluence/display/MADLIB/MADlib+1.10
> >>
> >> *** Please download, review and vote by Sat Feb 18, 2017 @ 6pm PST ***
> >>
> >> We're voting upon the source (tag): rc/1.10.0-rc1
> >> https://github.com/apache/incubator-madlib/tree/rc/1.10.0-rc1
> >>
> >> Source Files:
> >> https://dist.apache.org/repos/dist/dev/incubator/madlib/1.
> >> 10.0-incubating-rc1/
> >>
> >> Commit to be voted upon:
> >> https://github.com/apache/incubator-madlib/commit/
> >> ea17530bfe22a1fde173d7fa83508cbcd9924c20
> >>
> >> KEYS file containing PGP Keys we use to sign the release:
> >> https://dist.apache.org/repos/dist/dev/incubator/madlib/KEYS
> >>
> >> To help in tallying the vote, can PMC members please be sure to
> >> indicate "(binding)" with their vote.
> >>
> >> [ ] +1 approve
> >> [ ] +0 no opinion
> >> [ ] -1 disapprove (and reason why)
> >>
> >> --
> >> Satoshi Nagayasu <sn...@uptime.jp>
> >>
>
Re: [VOTE] MADlib v1.10-rc1
Posted by Orhan Kislal <ok...@pivotal.io>.
+1
Orhan Kislal
On Thu, Feb 16, 2017 at 9:23 AM, Joe Hellerstein <he...@berkeley.edu>
wrote:
> +1
>
> Sent from a telephone.
>
> > On Feb 16, 2017, at 9:17 AM, Frank McQuillan <fm...@pivotal.io>
> wrote:
> >
> > +1
> >
> > Frank McQuillan
> >
> >> On Wed, Feb 15, 2017 at 7:27 PM, Satoshi Nagayasu <sn...@uptime.jp>
> wrote:
> >>
> >> Hello MADlib community,
> >>
> >> We have created a MADlib 1.10 RC-1, with the artifacts below up for a
> vote.
> >>
> >> This will be the 4th release for Apache MADlib (incubating).
> >>
> >> The main goals of this release are:
> >> * new modules (single source shortest path for graph analytics, encode
> >> categorical variables, K-nearest neighbors)
> >> * improvements to existing modules (add grouping support to elastic
> >> net and PCA, add cross validation to elastic net, array input for
> >> K-means, verbose output option for DT and RF, limit itemset size in
> >> association rules, various madpack installer improvements)
> >> * platform updates (PostgreSQL 9.6)
> >> * bug fixes
> >> * doc improvements
> >>
> >> For more information including release notes, please see:
> >> https://cwiki.apache.org/confluence/display/MADLIB/MADlib+1.10
> >>
> >> *** Please download, review and vote by Sat Feb 18, 2017 @ 6pm PST ***
> >>
> >> We're voting upon the source (tag): rc/1.10.0-rc1
> >> https://github.com/apache/incubator-madlib/tree/rc/1.10.0-rc1
> >>
> >> Source Files:
> >> https://dist.apache.org/repos/dist/dev/incubator/madlib/1.
> >> 10.0-incubating-rc1/
> >>
> >> Commit to be voted upon:
> >> https://github.com/apache/incubator-madlib/commit/
> >> ea17530bfe22a1fde173d7fa83508cbcd9924c20
> >>
> >> KEYS file containing PGP Keys we use to sign the release:
> >> https://dist.apache.org/repos/dist/dev/incubator/madlib/KEYS
> >>
> >> To help in tallying the vote, can PMC members please be sure to
> >> indicate "(binding)" with their vote.
> >>
> >> [ ] +1 approve
> >> [ ] +0 no opinion
> >> [ ] -1 disapprove (and reason why)
> >>
> >> --
> >> Satoshi Nagayasu <sn...@uptime.jp>
> >>
>
Re: [VOTE] MADlib v1.10-rc1
Posted by Joe Hellerstein <he...@berkeley.edu>.
+1
Sent from a telephone.
> On Feb 16, 2017, at 9:17 AM, Frank McQuillan <fm...@pivotal.io> wrote:
>
> +1
>
> Frank McQuillan
>
>> On Wed, Feb 15, 2017 at 7:27 PM, Satoshi Nagayasu <sn...@uptime.jp> wrote:
>>
>> Hello MADlib community,
>>
>> We have created a MADlib 1.10 RC-1, with the artifacts below up for a vote.
>>
>> This will be the 4th release for Apache MADlib (incubating).
>>
>> The main goals of this release are:
>> * new modules (single source shortest path for graph analytics, encode
>> categorical variables, K-nearest neighbors)
>> * improvements to existing modules (add grouping support to elastic
>> net and PCA, add cross validation to elastic net, array input for
>> K-means, verbose output option for DT and RF, limit itemset size in
>> association rules, various madpack installer improvements)
>> * platform updates (PostgreSQL 9.6)
>> * bug fixes
>> * doc improvements
>>
>> For more information including release notes, please see:
>> https://cwiki.apache.org/confluence/display/MADLIB/MADlib+1.10
>>
>> *** Please download, review and vote by Sat Feb 18, 2017 @ 6pm PST ***
>>
>> We're voting upon the source (tag): rc/1.10.0-rc1
>> https://github.com/apache/incubator-madlib/tree/rc/1.10.0-rc1
>>
>> Source Files:
>> https://dist.apache.org/repos/dist/dev/incubator/madlib/1.
>> 10.0-incubating-rc1/
>>
>> Commit to be voted upon:
>> https://github.com/apache/incubator-madlib/commit/
>> ea17530bfe22a1fde173d7fa83508cbcd9924c20
>>
>> KEYS file containing PGP Keys we use to sign the release:
>> https://dist.apache.org/repos/dist/dev/incubator/madlib/KEYS
>>
>> To help in tallying the vote, can PMC members please be sure to
>> indicate "(binding)" with their vote.
>>
>> [ ] +1 approve
>> [ ] +0 no opinion
>> [ ] -1 disapprove (and reason why)
>>
>> --
>> Satoshi Nagayasu <sn...@uptime.jp>
>>
Re: [VOTE] MADlib v1.10-rc1
Posted by Marshall Presser <mp...@pivotal.io>.
+1
On Thu, Feb 16, 2017 at 12:17 PM, Frank McQuillan <fm...@pivotal.io>
wrote:
> +1
>
> Frank McQuillan
>
> On Wed, Feb 15, 2017 at 7:27 PM, Satoshi Nagayasu <sn...@uptime.jp> wrote:
>
>> Hello MADlib community,
>>
>> We have created a MADlib 1.10 RC-1, with the artifacts below up for a
>> vote.
>>
>> This will be the 4th release for Apache MADlib (incubating).
>>
>> The main goals of this release are:
>> * new modules (single source shortest path for graph analytics, encode
>> categorical variables, K-nearest neighbors)
>> * improvements to existing modules (add grouping support to elastic
>> net and PCA, add cross validation to elastic net, array input for
>> K-means, verbose output option for DT and RF, limit itemset size in
>> association rules, various madpack installer improvements)
>> * platform updates (PostgreSQL 9.6)
>> * bug fixes
>> * doc improvements
>>
>> For more information including release notes, please see:
>> https://cwiki.apache.org/confluence/display/MADLIB/MADlib+1.10
>>
>> *** Please download, review and vote by Sat Feb 18, 2017 @ 6pm PST ***
>>
>> We're voting upon the source (tag): rc/1.10.0-rc1
>> https://github.com/apache/incubator-madlib/tree/rc/1.10.0-rc1
>>
>> Source Files:
>> https://dist.apache.org/repos/dist/dev/incubator/madlib/1.10
>> .0-incubating-rc1/
>>
>> Commit to be voted upon:
>> https://github.com/apache/incubator-madlib/commit/ea17530bfe
>> 22a1fde173d7fa83508cbcd9924c20
>>
>> KEYS file containing PGP Keys we use to sign the release:
>> https://dist.apache.org/repos/dist/dev/incubator/madlib/KEYS
>>
>> To help in tallying the vote, can PMC members please be sure to
>> indicate "(binding)" with their vote.
>>
>> [ ] +1 approve
>> [ ] +0 no opinion
>> [ ] -1 disapprove (and reason why)
>>
>> --
>> Satoshi Nagayasu <sn...@uptime.jp>
>>
>
>
--
Marshall Presser
Pivotal Data Engineering
mpresser@pivotal <mp...@gopivotal.com>.io
240.401.1750
Re: [VOTE] MADlib v1.10-rc1
Posted by Marshall Presser <mp...@pivotal.io>.
+1
On Thu, Feb 16, 2017 at 12:17 PM, Frank McQuillan <fm...@pivotal.io>
wrote:
> +1
>
> Frank McQuillan
>
> On Wed, Feb 15, 2017 at 7:27 PM, Satoshi Nagayasu <sn...@uptime.jp> wrote:
>
>> Hello MADlib community,
>>
>> We have created a MADlib 1.10 RC-1, with the artifacts below up for a
>> vote.
>>
>> This will be the 4th release for Apache MADlib (incubating).
>>
>> The main goals of this release are:
>> * new modules (single source shortest path for graph analytics, encode
>> categorical variables, K-nearest neighbors)
>> * improvements to existing modules (add grouping support to elastic
>> net and PCA, add cross validation to elastic net, array input for
>> K-means, verbose output option for DT and RF, limit itemset size in
>> association rules, various madpack installer improvements)
>> * platform updates (PostgreSQL 9.6)
>> * bug fixes
>> * doc improvements
>>
>> For more information including release notes, please see:
>> https://cwiki.apache.org/confluence/display/MADLIB/MADlib+1.10
>>
>> *** Please download, review and vote by Sat Feb 18, 2017 @ 6pm PST ***
>>
>> We're voting upon the source (tag): rc/1.10.0-rc1
>> https://github.com/apache/incubator-madlib/tree/rc/1.10.0-rc1
>>
>> Source Files:
>> https://dist.apache.org/repos/dist/dev/incubator/madlib/1.10
>> .0-incubating-rc1/
>>
>> Commit to be voted upon:
>> https://github.com/apache/incubator-madlib/commit/ea17530bfe
>> 22a1fde173d7fa83508cbcd9924c20
>>
>> KEYS file containing PGP Keys we use to sign the release:
>> https://dist.apache.org/repos/dist/dev/incubator/madlib/KEYS
>>
>> To help in tallying the vote, can PMC members please be sure to
>> indicate "(binding)" with their vote.
>>
>> [ ] +1 approve
>> [ ] +0 no opinion
>> [ ] -1 disapprove (and reason why)
>>
>> --
>> Satoshi Nagayasu <sn...@uptime.jp>
>>
>
>
--
Marshall Presser
Pivotal Data Engineering
mpresser@pivotal <mp...@gopivotal.com>.io
240.401.1750
Re: [VOTE] MADlib v1.10-rc1
Posted by Frank McQuillan <fm...@pivotal.io>.
+1
Frank McQuillan
On Wed, Feb 15, 2017 at 7:27 PM, Satoshi Nagayasu <sn...@uptime.jp> wrote:
> Hello MADlib community,
>
> We have created a MADlib 1.10 RC-1, with the artifacts below up for a vote.
>
> This will be the 4th release for Apache MADlib (incubating).
>
> The main goals of this release are:
> * new modules (single source shortest path for graph analytics, encode
> categorical variables, K-nearest neighbors)
> * improvements to existing modules (add grouping support to elastic
> net and PCA, add cross validation to elastic net, array input for
> K-means, verbose output option for DT and RF, limit itemset size in
> association rules, various madpack installer improvements)
> * platform updates (PostgreSQL 9.6)
> * bug fixes
> * doc improvements
>
> For more information including release notes, please see:
> https://cwiki.apache.org/confluence/display/MADLIB/MADlib+1.10
>
> *** Please download, review and vote by Sat Feb 18, 2017 @ 6pm PST ***
>
> We're voting upon the source (tag): rc/1.10.0-rc1
> https://github.com/apache/incubator-madlib/tree/rc/1.10.0-rc1
>
> Source Files:
> https://dist.apache.org/repos/dist/dev/incubator/madlib/1.
> 10.0-incubating-rc1/
>
> Commit to be voted upon:
> https://github.com/apache/incubator-madlib/commit/
> ea17530bfe22a1fde173d7fa83508cbcd9924c20
>
> KEYS file containing PGP Keys we use to sign the release:
> https://dist.apache.org/repos/dist/dev/incubator/madlib/KEYS
>
> To help in tallying the vote, can PMC members please be sure to
> indicate "(binding)" with their vote.
>
> [ ] +1 approve
> [ ] +0 no opinion
> [ ] -1 disapprove (and reason why)
>
> --
> Satoshi Nagayasu <sn...@uptime.jp>
>
Re: [VOTE] MADlib v1.10-rc1
Posted by Rahul Iyer <ra...@gmail.com>.
+1
On Wed, Feb 15, 2017 at 7:27 PM, Satoshi Nagayasu <sn...@uptime.jp> wrote:
> Hello MADlib community,
>
> We have created a MADlib 1.10 RC-1, with the artifacts below up for a vote.
>
> This will be the 4th release for Apache MADlib (incubating).
>
> The main goals of this release are:
> * new modules (single source shortest path for graph analytics, encode
> categorical variables, K-nearest neighbors)
> * improvements to existing modules (add grouping support to elastic
> net and PCA, add cross validation to elastic net, array input for
> K-means, verbose output option for DT and RF, limit itemset size in
> association rules, various madpack installer improvements)
> * platform updates (PostgreSQL 9.6)
> * bug fixes
> * doc improvements
>
> For more information including release notes, please see:
> https://cwiki.apache.org/confluence/display/MADLIB/MADlib+1.10
>
> *** Please download, review and vote by Sat Feb 18, 2017 @ 6pm PST ***
>
> We're voting upon the source (tag): rc/1.10.0-rc1
> https://github.com/apache/incubator-madlib/tree/rc/1.10.0-rc1
>
> Source Files:
> https://dist.apache.org/repos/dist/dev/incubator/madlib/1.
> 10.0-incubating-rc1/
>
> Commit to be voted upon:
> https://github.com/apache/incubator-madlib/commit/
> ea17530bfe22a1fde173d7fa83508cbcd9924c20
>
> KEYS file containing PGP Keys we use to sign the release:
> https://dist.apache.org/repos/dist/dev/incubator/madlib/KEYS
>
> To help in tallying the vote, can PMC members please be sure to
> indicate "(binding)" with their vote.
>
> [ ] +1 approve
> [ ] +0 no opinion
> [ ] -1 disapprove (and reason why)
>
> --
> Satoshi Nagayasu <sn...@uptime.jp>
>
Re: [VOTE] MADlib v1.10-rc1
Posted by Rahul Iyer <ra...@gmail.com>.
+1
On Wed, Feb 15, 2017 at 7:27 PM, Satoshi Nagayasu <sn...@uptime.jp> wrote:
> Hello MADlib community,
>
> We have created a MADlib 1.10 RC-1, with the artifacts below up for a vote.
>
> This will be the 4th release for Apache MADlib (incubating).
>
> The main goals of this release are:
> * new modules (single source shortest path for graph analytics, encode
> categorical variables, K-nearest neighbors)
> * improvements to existing modules (add grouping support to elastic
> net and PCA, add cross validation to elastic net, array input for
> K-means, verbose output option for DT and RF, limit itemset size in
> association rules, various madpack installer improvements)
> * platform updates (PostgreSQL 9.6)
> * bug fixes
> * doc improvements
>
> For more information including release notes, please see:
> https://cwiki.apache.org/confluence/display/MADLIB/MADlib+1.10
>
> *** Please download, review and vote by Sat Feb 18, 2017 @ 6pm PST ***
>
> We're voting upon the source (tag): rc/1.10.0-rc1
> https://github.com/apache/incubator-madlib/tree/rc/1.10.0-rc1
>
> Source Files:
> https://dist.apache.org/repos/dist/dev/incubator/madlib/1.
> 10.0-incubating-rc1/
>
> Commit to be voted upon:
> https://github.com/apache/incubator-madlib/commit/
> ea17530bfe22a1fde173d7fa83508cbcd9924c20
>
> KEYS file containing PGP Keys we use to sign the release:
> https://dist.apache.org/repos/dist/dev/incubator/madlib/KEYS
>
> To help in tallying the vote, can PMC members please be sure to
> indicate "(binding)" with their vote.
>
> [ ] +1 approve
> [ ] +0 no opinion
> [ ] -1 disapprove (and reason why)
>
> --
> Satoshi Nagayasu <sn...@uptime.jp>
>
Re: [VOTE] MADlib v1.10-rc1
Posted by Frank McQuillan <fm...@pivotal.io>.
+1
Frank McQuillan
On Wed, Feb 15, 2017 at 7:27 PM, Satoshi Nagayasu <sn...@uptime.jp> wrote:
> Hello MADlib community,
>
> We have created a MADlib 1.10 RC-1, with the artifacts below up for a vote.
>
> This will be the 4th release for Apache MADlib (incubating).
>
> The main goals of this release are:
> * new modules (single source shortest path for graph analytics, encode
> categorical variables, K-nearest neighbors)
> * improvements to existing modules (add grouping support to elastic
> net and PCA, add cross validation to elastic net, array input for
> K-means, verbose output option for DT and RF, limit itemset size in
> association rules, various madpack installer improvements)
> * platform updates (PostgreSQL 9.6)
> * bug fixes
> * doc improvements
>
> For more information including release notes, please see:
> https://cwiki.apache.org/confluence/display/MADLIB/MADlib+1.10
>
> *** Please download, review and vote by Sat Feb 18, 2017 @ 6pm PST ***
>
> We're voting upon the source (tag): rc/1.10.0-rc1
> https://github.com/apache/incubator-madlib/tree/rc/1.10.0-rc1
>
> Source Files:
> https://dist.apache.org/repos/dist/dev/incubator/madlib/1.
> 10.0-incubating-rc1/
>
> Commit to be voted upon:
> https://github.com/apache/incubator-madlib/commit/
> ea17530bfe22a1fde173d7fa83508cbcd9924c20
>
> KEYS file containing PGP Keys we use to sign the release:
> https://dist.apache.org/repos/dist/dev/incubator/madlib/KEYS
>
> To help in tallying the vote, can PMC members please be sure to
> indicate "(binding)" with their vote.
>
> [ ] +1 approve
> [ ] +0 no opinion
> [ ] -1 disapprove (and reason why)
>
> --
> Satoshi Nagayasu <sn...@uptime.jp>
>
Re: [VOTE] MADlib v1.10-rc1
Posted by Roman Shaposhnik <ro...@shaposhnik.org>.
On Wed, Feb 15, 2017 at 7:27 PM, Satoshi Nagayasu <sn...@uptime.jp> wrote:
> Hello MADlib community,
>
> We have created a MADlib 1.10 RC-1, with the artifacts below up for a vote.
>
> This will be the 4th release for Apache MADlib (incubating).
>
> The main goals of this release are:
> * new modules (single source shortest path for graph analytics, encode
> categorical variables, K-nearest neighbors)
> * improvements to existing modules (add grouping support to elastic
> net and PCA, add cross validation to elastic net, array input for
> K-means, verbose output option for DT and RF, limit itemset size in
> association rules, various madpack installer improvements)
> * platform updates (PostgreSQL 9.6)
> * bug fixes
> * doc improvements
>
> For more information including release notes, please see:
> https://cwiki.apache.org/confluence/display/MADLIB/MADlib+1.10
>
> *** Please download, review and vote by Sat Feb 18, 2017 @ 6pm PST ***
>
> We're voting upon the source (tag): rc/1.10.0-rc1
> https://github.com/apache/incubator-madlib/tree/rc/1.10.0-rc1
>
> Source Files:
> https://dist.apache.org/repos/dist/dev/incubator/madlib/1.10.0-incubating-rc1/
>
> Commit to be voted upon:
> https://github.com/apache/incubator-madlib/commit/ea17530bfe22a1fde173d7fa83508cbcd9924c20
>
> KEYS file containing PGP Keys we use to sign the release:
> https://dist.apache.org/repos/dist/dev/incubator/madlib/KEYS
>
> To help in tallying the vote, can PMC members please be sure to
> indicate "(binding)" with their vote.
>
> [ ] +1 approve
> [ ] +0 no opinion
> [ ] -1 disapprove (and reason why)
>
> --
> Satoshi Nagayasu <sn...@uptime.jp>
I'm afraid, I'll have to be -1
It seems that the following files were not a straight rename:
src/ports/postgres/modules/utilities/data_preparation.py_in ->
src/ports/postgres/modules/utilities/create_indicators.py_in
src/ports/postgres/modules/utilities/data_preparation.sql_in ->
src/ports/postgres/modules/utilities/create_indicators.sql_in
but in fact were modified quite a bit. As such we should really treat those
as new work that is now licensed under the ALv2 and requires ALv2
license header.
I know this dual situation with BSD/ALv2 is not ideal and I'm trying to
resolve it for us once and for all so we don't have to worry about things
like this one:
https://issues.apache.org/jira/browse/LEGAL-293
Since we'll have to do a re-spin, I'd like to also suggest that we take
care of the nits in Ed's feedback.
Hope this is OK with Satoshi.
Thanks,
Roman.