John, is there any additional information we can provide for your review? Frank
On Wed, Sep 7, 2016 at 6:13 PM, Orhan Kislal <okis...@pivotal.io> wrote: > John, > > I know of this commit and I cannot see any distinction between the > src/utils folder and many other excluded folders. Here is the phrase from > the LICENSE file: > > The rest of the source code, unless explicitly marked with an Apache > License > header, should be assumed to be coming from previous life of MADlib as a > BSD licensed project and is available under the following license: > ... > > Thanks, > > Orhan > > On Wed, Sep 7, 2016 at 6:00 PM, John D. Ament <johndam...@apache.org> > wrote: > > > Frank pointed me to this commit - which changes a bunch of things to > Apache > > License: > > https://github.com/apache/incubator-madlib/commit/ > > a6acafdd54bd5e5d53ac3b7fb4b6621c042af460#diff- > > 600376dffeb79835ede4a0b285078036 > > > > John > > > > On Wed, Sep 7, 2016 at 8:58 PM Orhan Kislal <okis...@pivotal.io> wrote: > > > > > John, > > > > > > I am checking MADlib github page for the src/utils folder and it > > indicates > > > that the latest commits are from 4-5 years ago. Could you point us to > the > > > commit tag that you mentioned? > > > > > > Thanks, > > > > > > Orhan > > > > > > On Wed, Sep 7, 2016 at 5:47 PM, John D. Ament <johndam...@apache.org> > > > wrote: > > > > > > > Frank, > > > > > > > > Then those files should have Apache headers (they're not listed in > the > > > > explicitly licensed area & are part of a commit that added apache > > > > headers). Can you explain why you think they're not subject, or what > > > > license they're falling under? > > > > > > > > John > > > > > > > > On Wed, Sep 7, 2016 at 8:41 PM Frank McQuillan < > fmcquil...@pivotal.io> > > > > wrote: > > > > > > > > > John, > > > > > > > > > > src/utils files were in the original commit by Roman when we > brought > > > the > > > > > files into ASF: > > > > > > > > > > https://github.com/apache/incubator-madlib/commit/ > > > > a6acafdd54bd5e5d53ac3b7fb4b6621c042af460#diff- > > > > 600376dffeb79835ede4a0b285078036 > > > > > and they have not been changed since that time. > > > > > > > > > > tag: a6acafdd54bd5e5d53ac3b7fb4b6621c042af460 > > > > > > > > > > So I do not believe any changes to NOTICE/LICENSE are required for > > > > > src/utils > > > > > > > > > > Frank > > > > > > > > > > > > > > > > > > > > On Wed, Sep 7, 2016 at 5:28 PM, John D. Ament < > johndam...@apache.org > > > > > > > > wrote: > > > > > > > > > > > On Wed, Sep 7, 2016 at 8:01 PM Frank McQuillan < > > > fmcquil...@pivotal.io> > > > > > > wrote: > > > > > > > > > > > > > Hi John, > > > > > > > > > > > > > > Thanks for reviewing the release. > > > > > > > > > > > > > > Regarding the exclusion list, during the first Apache incubator > > > > release > > > > > > for > > > > > > > MADlib 1.9 alpha, there was a legal-discuss thread on how to > > > proceed > > > > > with > > > > > > > headers that came from pre-Apache days when MADlib was under a > > > > 2-clause > > > > > > BSD > > > > > > > license. > > > > > > > > > > > > > > The agreement was to add them to the exclusion list as per: > > > > > > > http://mail-archives.apache.org/mod_mbox/www-legal-discuss/ > > > > > > > > > > 201603.mbox/%3C9D1AF43C-370B-4E58-B0EF-2E29D242F50B%40jaguNET.com%3E > > > > > > > which we have done for the prior 2 releases. So for this 3rd > > > release > > > > > we > > > > > > > did the same. > > > > > > > > > > > > > > > > > > > Ok, that clarifies part of it. I don't see a call out for > > src/utils. > > > > > Why > > > > > > are those in the exclusion list and will you be updating > > > > NOTICE/LICENSE? > > > > > > > > > > > > > > > > > > > > > > > > > > Please let me know if you have any questions on this. > > > > > > > > > > > > > > Frank > > > > > > > > > > > > > > On Wed, Sep 7, 2016 at 4:18 PM, Rahul Iyer < > rahulri...@gmail.com > > > > > > > > wrote: > > > > > > > > > > > > > > > Hi John, > > > > > > > > > > > > > > > > We've uploaded the KEYS file to apache dist > > > > > > > > <https://dist.apache.org/repos/dist/dev/incubator/ > madlib/KEYS> > > > and > > > > > > > didn't > > > > > > > > use any public server for the previous 2 releases. > > > > > > > > > > > > > > > > I've uploaded my key (which was used for the signature) to > > > > > > > keyservers.net > > > > > > > > and should be available now after you update your local > cache. > > Is > > > > > this > > > > > > > the > > > > > > > > Apache recommended server to use? > > > > > > > > > > > > > > > > - Rahul > > > > > > > > > > > > > > > > On Wed, Sep 7, 2016 at 1:53 PM, John D. Ament < > > > > johndam...@apache.org > > > > > > > > > > > > > > wrote: > > > > > > > > > > > > > > > > > I'm getting a problem with your key. Can you look at it > and > > > tell > > > > > me > > > > > > if > > > > > > > > you > > > > > > > > > think the issue's on my end? > > > > > > > > > > > > > > > > > > Johns-MBP-2:Downloads johnament$ gpg --verify > > > > > > > > > apache-madlib-1.9.1-incubating-source.tar.gz.asc > > > > > > > > > gpg: assuming signed data in 'apache-madlib-1.9.1- > > > > > > > > > incubating-source.tar.gz' > > > > > > > > > gpg: Signature made Fri Sep 2 12:05:14 2016 EDT using RSA > > key > > > ID > > > > > > > > 6C725F40 > > > > > > > > > gpg: requesting key 6C725F40 from hkps server > > > > > > > > hkps.pool.sks-keyservers.net > > > > > > > > > gpgkeys: key 52B795C96C725F40 not found on keyserver > > > > > > > > > gpg: no valid OpenPGP data found. > > > > > > > > > gpg: Total number processed: 0 > > > > > > > > > gpg: keyserver communications error: Not found > > > > > > > > > gpg: keyserver communications error: Bad public key > > > > > > > > > gpg: Can't check signature: No public key > > > > > > > > > > > > > > > > > > Do I need to import your KEYS file or is your key uploaded > > to a > > > > > > public > > > > > > > > > server? > > > > > > > > > > > > > > > > > > John > > > > > > > > > > > > > > > > > > On Wed, Sep 7, 2016 at 12:40 AM Frank McQuillan < > > > > > > fmcquil...@pivotal.io > > > > > > > > > > > > > > > > > wrote: > > > > > > > > > > > > > > > > > > > Hello Incubator PMC, > > > > > > > > > > > > > > > > > > > > The Apache MADlib (incubating) community has voted on and > > > > > approved > > > > > > > the > > > > > > > > > > proposal to release MADlib v1.9.1-rc2. > > > > > > > > > > > > > > > > > > > > The voting result is available at: > > > > > > > > > > > > > > > > > > > > https://mail-archives.apache. > > org/mod_mbox/incubator-madlib- > > > > > > > > > dev/201609.mbox/%3CCAKBQfzT0pw1%3DP- > > KWuZVsAwORdHdz792WvUtR-DbM% > > > > > > > > > 2BH8EkiZdSA%40mail.gmail.com%3E > > > > > > > > > > > > > > > > > > > > This is the 3rd release for Apache MADlib (incubating). > > > > > > > > > > > > > > > > > > > > The main goals of this release are: > > > > > > > > > > * new modules (1-class SVM for novelty detection, > > prediction > > > > > > metrics, > > > > > > > > > > sessionization, pivoting) > > > > > > > > > > * improvements to existing modules (class weights in SVM, > > > > > > overlapping > > > > > > > > > > patterns in path) > > > > > > > > > > * performance improvements (path) > > > > > > > > > > * platform updates (PostgreSQL 9.5 and 9.6) > > > > > > > > > > * bug fixes > > > > > > > > > > * doc improvements > > > > > > > > > > > > > > > > > > > > For more information including release notes, please see: > > > > > > > > > > https://cwiki.apache.org/confluence/display/MADLIB/ > > > > MADlib+1.9.1 > > > > > > > > > > > > > > > > > > > > To run check RAT, please do: > > > > > > > > > > > > > > > > > > > > $mvn verify > > > > > > > > > > > > > > > > > > > > first to get the correct RAT output. Look inside of > > pom.xml > > > to > > > > > see > > > > > > > the > > > > > > > > > > classes of exceptions we're managing there for RAT. > > > > > > > > > > > > > > > > > > > > We're voting on the source (tag): > > > > > > > > > > rc/1.9.1-rc2 > > > > > > > > > > > > > > > > > > > > Source Files: > > > > > > > > > > > > > > > > > > > > https://dist.apache.org/repos/ > > dist/dev/incubator/madlib/1.9. > > > > > > > > > 1-incubating-rc2 > > > > > > > > > > > > > > > > > > > > Commit to be voted upon: > > > > > > > > > > > > > > > > > > > > https://git-wip-us.apache.org/ > repos/asf?p=incubator-madlib > > . > > > > > > > > > git;a=commit;h=e1c99c1538dc124c9b323ba76382ba2af05c6892 > > > > > > > > > > > > > > > > > > > > KEYS file containing PGP Keys we use to sign the release: > > > > > > > > > > https://dist.apache.org/repos/ > > dist/dev/incubator/madlib/KEYS > > > > > > > > > > > > > > > > > > > > Please vote: > > > > > > > > > > > > > > > > > > > > [ ] +1 approve > > > > > > > > > > [ ] +0 no opinion > > > > > > > > > > [ ] -1 disapprove (and reason why) > > > > > > > > > > > > > > > > > > > > *** The vote will be open until Friday Sept 9 at 6 pm > > Pacific > > > > > time. > > > > > > > *** > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > >
