You are viewing a plain text version of this content. The canonical link for it is here.
Posted to general@incubator.apache.org by "John D. Ament" <jo...@apache.org> on 2016/07/01 03:41:50 UTC

Re: [VOTE] Release Apache Trafodion (incubating) release 2.0.1-incubating RC3

+1 on the release.
Contents look correct.

One question though, are you 100% certain that the replaced license is
correct? Are there any other changes in the impacted files that would
require rework, or is the file stable enough that its not the case?

John

On Wed, Jun 29, 2016 at 5:52 PM Roberta Marton <ro...@esgyn.com>
wrote:

> Hello,
>
>
>
> This is a call to vote on Apache Trafodion patch release 2.0.1-incubating
> RC3 (Release Candidate 3).
>
>
>
> Apache Trafodion is a webscale SQL-on-Hadoop solution enabling
> transactional or
>
> operational workloads on Hadoop.  Trafodion builds on the scalability,
> elasticity, and
>
> flexibility of Hadoop. Trafodion extends Hadoop to provide guaranteed
>
> transactional integrity, enabling new kinds of big data applications to run
> on
>
> Hadoop.
>
>
>
> The changes in this patch release address licensing issues found during the
>
> Apache Trafodion release 2.0.1 RC1 (Release Candidate 1) voting cycle.
>
>
>
> Changes since 2.0.1 RC1:
>
>    [TRAFODION-2068] Missing DISCLAIMER files for release package
>
>    [TRAFODION-2082] BSD-4 license for swsprintf and swscanf unclear
>
>
>
> The Trafodion community has approved  this release:
>
>
> http://mail-archives.apache.org/mod_mbox/incubator-trafodion-dev/201606.mbox/ajax/%3Cab0bbe9228d5b7b487f4b9952f17ede5%40mail.gmail.com%3E
>
>
>
> The latest tag for this candidate is "2.0.1rc3". Git repository: git://
> git.apache.org/incubator-trafodion.git:
>
> *
> https://git-wip-us.apache.org/repos/asf?p=incubator-trafodion.git;a=commit;h=6357cb8f76ab399b451896c75273fd6f2e02d9c4
> <
> https://git-wip-us.apache.org/repos/asf?p=incubator-trafodion.git;a=commit;h=6357cb8f76ab399b451896c75273fd6f2e02d9c4
> >*
>
>
>
> Release artifacts are:
>
> *
> https://dist.apache.org/repos/dist/dev/incubator/trafodion/trafodion-2.0.1-RC3
> <
> https://dist.apache.org/repos/dist/dev/incubator/trafodion/trafodion-2.0.1-RC3
> >*
>
>
>
> Artifacts are signed with my key (A44C5A05), which is in
> https://dist.apache.org/repos/dist/release/incubator/trafodion/KEYS
>
>
>
> Instructions:
>
>
>
> Setting up build environment:
>
> *
> https://cwiki.apache.org/confluence/display/TRAFODION/Create+Build+Environment
> <
> https://cwiki.apache.org/confluence/display/TRAFODION/Create+Build+Environment
> >*
>
>
>
> Building:
>
> https://cwiki.apache.org/confluence/display/TRAFODION/Build+Source
>
>
>
> The vote is open for at least 72 hours.
>
>
>
> [ ] +1 approve
>
> [ ] +0 no opinion
>
> [ ] -1 disapprove (and reason why)
>
>
>
>   Regards,
>
>   Roberta
>

RE: [VOTE] Release Apache Trafodion (incubating) release 2.0.1-incubating RC3

Posted by Roberta Marton <ro...@esgyn.com>.
Thanks Justin for your review, I have filed JIRA's TRAFODION-2092 and
TRAFODION-2093 to fix the issues you mentioned.

    Regards,
    Roberta

-----Original Message-----
From: Justin Mclean [mailto:justin@classsoftware.com]
Sent: Saturday, July 2, 2016 4:51 PM
To: general@incubator.apache.org
Subject: Re: [VOTE] Release Apache Trafodion (incubating) release
2.0.1-incubating RC3

Hi,

1+ binding

I checked:
- name include incubating (except installer)
- signatures and hashes good
- DISCLAIMER exists
- LICENSE and NOTICE good
- no unexpected binary files in source release
- all source files have ASF headers

Is there any reason for the installer to not include apache-trafodion and
incubating? Please fix for next release.

I’m also a little confused by “trafodion_license” in the installer as it
states:
“Project Trafodion includes third party copyrighted software components,
which may be licensed under third party and or open source license terms as
set forth in the LICENSE.TXT file accompanying the Project Trafodion
distribution or as indicated in the applicable source code file.”

But LICENSE (not LICENSE.TXT) has no 3rd party software. Is the
trafodion_license file even needed?

With the addition of the new 3 clause BSD licenses the number “2” is the
second license point has been dropped, you might want to fix that.

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

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


Re: [VOTE] Release Apache Trafodion (incubating) release 2.0.1-incubating RC3

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

1+ binding

I checked:
- name include incubating (except installer)
- signatures and hashes good
- DISCLAIMER exists
- LICENSE and NOTICE good
- no unexpected binary files in source release
- all source files have ASF headers

Is there any reason for the installer to not include apache-trafodion and incubating? Please fix for next release.

I’m also a little confused by “trafodion_license” in the installer as it states:
“Project Trafodion includes third party copyrighted software
components, which may be licensed under third party and or
open source license terms as set forth in the LICENSE.TXT
file accompanying the Project Trafodion distribution or as
indicated in the applicable source code file.”

But LICENSE (not LICENSE.TXT) has no 3rd party software. Is the trafodion_license file even needed?

With the addition of the new 3 clause BSD licenses the number “2” is the second license point has been dropped, you might want to fix that.

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


Re: [VOTE] Release Apache Trafodion (incubating) release 2.0.1-incubating RC3

Posted by Roberta Marton <ro...@esgyn.com>.
Thanks for your vote. Appreciate you taking time to review the release.

The original code included a website where the code was downloaded from but
the address was no longer valid. So I found a version with the BSD 3
license. I downloaded the files and compared them to Trafodion files. I was
concerned that they might be different and that Trafodion may have made
enough changes to deserve a separate copyright.

The files did have several differences, for example, Trafodion changed some
of the method names but the basic logic was the same. So I felt that I
could replace the license and that we didn't need a separate copyright;
that is, the files were similar enough. These files are stable.

I also updated the web site location in the code to point to the BSD 3
files.

   Roberta
On Jun 30, 2016 8:42 PM, "John D. Ament" <jo...@apache.org> wrote:

> +1 on the release.
> Contents look correct.
>
> One question though, are you 100% certain that the replaced license is
> correct? Are there any other changes in the impacted files that would
> require rework, or is the file stable enough that its not the case?
>
> John
>
> On Wed, Jun 29, 2016 at 5:52 PM Roberta Marton <ro...@esgyn.com>
> wrote:
>
> > Hello,
> >
> >
> >
> > This is a call to vote on Apache Trafodion patch release 2.0.1-incubating
> > RC3 (Release Candidate 3).
> >
> >
> >
> > Apache Trafodion is a webscale SQL-on-Hadoop solution enabling
> > transactional or
> >
> > operational workloads on Hadoop.  Trafodion builds on the scalability,
> > elasticity, and
> >
> > flexibility of Hadoop. Trafodion extends Hadoop to provide guaranteed
> >
> > transactional integrity, enabling new kinds of big data applications to
> run
> > on
> >
> > Hadoop.
> >
> >
> >
> > The changes in this patch release address licensing issues found during
> the
> >
> > Apache Trafodion release 2.0.1 RC1 (Release Candidate 1) voting cycle.
> >
> >
> >
> > Changes since 2.0.1 RC1:
> >
> >    [TRAFODION-2068] Missing DISCLAIMER files for release package
> >
> >    [TRAFODION-2082] BSD-4 license for swsprintf and swscanf unclear
> >
> >
> >
> > The Trafodion community has approved  this release:
> >
> >
> >
> http://mail-archives.apache.org/mod_mbox/incubator-trafodion-dev/201606.mbox/ajax/%3Cab0bbe9228d5b7b487f4b9952f17ede5%40mail.gmail.com%3E
> >
> >
> >
> > The latest tag for this candidate is "2.0.1rc3". Git repository: git://
> > git.apache.org/incubator-trafodion.git:
> >
> > *
> >
> https://git-wip-us.apache.org/repos/asf?p=incubator-trafodion.git;a=commit;h=6357cb8f76ab399b451896c75273fd6f2e02d9c4
> > <
> >
> https://git-wip-us.apache.org/repos/asf?p=incubator-trafodion.git;a=commit;h=6357cb8f76ab399b451896c75273fd6f2e02d9c4
> > >*
> >
> >
> >
> > Release artifacts are:
> >
> > *
> >
> https://dist.apache.org/repos/dist/dev/incubator/trafodion/trafodion-2.0.1-RC3
> > <
> >
> https://dist.apache.org/repos/dist/dev/incubator/trafodion/trafodion-2.0.1-RC3
> > >*
> >
> >
> >
> > Artifacts are signed with my key (A44C5A05), which is in
> > https://dist.apache.org/repos/dist/release/incubator/trafodion/KEYS
> >
> >
> >
> > Instructions:
> >
> >
> >
> > Setting up build environment:
> >
> > *
> >
> https://cwiki.apache.org/confluence/display/TRAFODION/Create+Build+Environment
> > <
> >
> https://cwiki.apache.org/confluence/display/TRAFODION/Create+Build+Environment
> > >*
> >
> >
> >
> > Building:
> >
> > https://cwiki.apache.org/confluence/display/TRAFODION/Build+Source
> >
> >
> >
> > The vote is open for at least 72 hours.
> >
> >
> >
> > [ ] +1 approve
> >
> > [ ] +0 no opinion
> >
> > [ ] -1 disapprove (and reason why)
> >
> >
> >
> >   Regards,
> >
> >   Roberta
> >
>