+1

NJ

Sent from my iPhone

> On Feb 16, 2017, at 12:06 PM, Orhan Kislal <okis...@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 <esp...@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 <okis...@pivotal.io> wrote:
>> 
>> > +1
>> >
>> > Orhan Kislal
>> >
>> > On Thu, Feb 16, 2017 at 9:23 AM, Joe Hellerstein <hellerst...@berkeley.edu
>> > >
>> > wrote:
>> >
>> > > +1
>> > >
>> > > Sent from a telephone.
>> > >
>> > > > On Feb 16, 2017, at 9:17 AM, Frank McQuillan <fmcquil...@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*
>> *esp...@apache.org <esp...@apache.org>*
> 

Reply via email to