You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@parquet.apache.org by Ryan Blue <bl...@apache.org> on 2015/04/08 01:57:53 UTC

[VOTE] Release Apache Parquet MR (Incubating) 1.6.0 RC2

Hi everyone,

I propose the following RC to be released as official Apache Parquet 
1.6.0 release.

This is the first Apache release of Parquet MR. Like the 2.2.0 release 
of Parquet Format, this uses the old parquet.* package names and will be 
immediately followed by a 1.7.0 release that is strictly a move to 
org.apache.parquet.* packages and artifacts.

The commit id is 4f660778f89d4164c72676f6b15bbc74c0d09373.
* http://s.apache.org/apache-parquet-mr-1.6.0-incubating-rc2
* https://github.com/apache/incubator-parquet-mr/tree/4f660778

The release tarball, signature, and checksums are here:
* 
https://dist.apache.org/repos/dist/dev/incubator/parquet/apache-parquet-mr-1.6.0-incubating-rc2/

You can find the KEYS file here:
* https://dist.apache.org/repos/dist/dev/incubator/parquet/KEYS

Full changes for the release can be found here:
* 
https://github.com/apache/incubator-parquet-mr/blob/4f660778/CHANGES.md#version-160

Please download, verify, and test.

This vote will close in 3 days, so please vote by Friday evening, PDT.

[ ] +1 Release this as Apache Parquet Format 1.6.0
[ ] +0
[ ] -1 Do not release this because...

-- 
Ryan Blue

[RESULT] [VOTE] Release Apache Parquet MR (Incubating) 1.6.0 RC2

Posted by Ryan Blue <bl...@cloudera.com>.
This release vote passes with 6 +1 votes and no -1 or 0 votes. Thanks 
for checking the release and voting, everyone! I'll start the IPMC vote 
thread.

rb

On 04/07/2015 04:57 PM, Ryan Blue wrote:
> Hi everyone,
>
> I propose the following RC to be released as official Apache Parquet
> 1.6.0 release.
>
> This is the first Apache release of Parquet MR. Like the 2.2.0 release
> of Parquet Format, this uses the old parquet.* package names and will be
> immediately followed by a 1.7.0 release that is strictly a move to
> org.apache.parquet.* packages and artifacts.
>
> The commit id is 4f660778f89d4164c72676f6b15bbc74c0d09373.
> * http://s.apache.org/apache-parquet-mr-1.6.0-incubating-rc2
> * https://github.com/apache/incubator-parquet-mr/tree/4f660778
>
> The release tarball, signature, and checksums are here:
> *
> https://dist.apache.org/repos/dist/dev/incubator/parquet/apache-parquet-mr-1.6.0-incubating-rc2/
>
>
> You can find the KEYS file here:
> * https://dist.apache.org/repos/dist/dev/incubator/parquet/KEYS
>
> Full changes for the release can be found here:
> *
> https://github.com/apache/incubator-parquet-mr/blob/4f660778/CHANGES.md#version-160
>
>
> Please download, verify, and test.
>
> This vote will close in 3 days, so please vote by Friday evening, PDT.
>
> [ ] +1 Release this as Apache Parquet Format 1.6.0
> [ ] +0
> [ ] -1 Do not release this because...
>


-- 
Ryan Blue
Software Engineer
Cloudera, Inc.

Re: [VOTE] Release Apache Parquet MR (Incubating) 1.6.0 RC2

Posted by Ryan Blue <bl...@cloudera.com>.
On 04/08/2015 10:09 AM, Tom White wrote:
> I cleared out the 1.5.0 Parquet jars from my local maven repo, and it passed. +1
>
> Cheers,
> Tom

Thanks Tom!

+1 from me as well, I checked the signature, checksums, and ran `mvn 
clean install` without problems.

rb

-- 
Ryan Blue
Software Engineer
Cloudera, Inc.

Re: [VOTE] Release Apache Parquet MR (Incubating) 1.6.0 RC2

Posted by Tom White <to...@cloudera.com>.
I cleared out the 1.5.0 Parquet jars from my local maven repo, and it passed. +1

Cheers,
Tom

On Wed, Apr 8, 2015 at 5:45 PM, Ryan Blue <bl...@cloudera.com> wrote:
> On 04/08/2015 07:23 AM, Tom White wrote:
>>
>> I checked signature and checksums, ran RAT to check license headers,
>> verified NOTICE, LICENSE and DISCLAIMER - all these look good.
>> However, when I build with 'mvn clean install' I get an enforcer
>> error:
>>
>> [WARNING] Rule 0:
>> org.semver.enforcer.RequireSemanticVersioningConformance failed with
>> message:
>>
>> Current codebase is incompatible with version <1.5.0>. Version should
>> be at least <2.0.0>.
>>
>> Tom
>
>
> If I run `mvn clean install` on the release candidate, it succeeds. I looked
> at your error gist [1] and I don't see one of the methods it complains
> about, `mergeKeyValueMetadata`, in 1.5.0 [2]. (The `init` method is provided
> by the ReadSupport, which semver can't catch.)
>
> Can you check your repository's copy of parquet-avro-1.5.0? My sha1 is this
> (which matches the .sha1 file):
>   2a44461074b5624a5d8892ce1daf33bc34207e00  .../parquet-avro-1.5.0.jar
>
> Looks like Travis CI is successful as well [3]. Can someone else try this?
>
> rb
>
> [1]: https://gist.github.com/tomwhite/5232b3e929d223618b5e
> [2]:
> https://github.com/apache/incubator-parquet-mr/blob/parquet-1.5.0/parquet-avro/src/main/java/parquet/avro/AvroReadSupport.java
> [3]: https://travis-ci.org/apache/incubator-parquet-mr/builds/57570537
>
>
> --
> Ryan Blue
> Software Engineer
> Cloudera, Inc.

Re: [VOTE] Release Apache Parquet MR (Incubating) 1.6.0 RC2

Posted by Ryan Blue <bl...@cloudera.com>.
On 04/08/2015 07:23 AM, Tom White wrote:
> I checked signature and checksums, ran RAT to check license headers,
> verified NOTICE, LICENSE and DISCLAIMER - all these look good.
> However, when I build with 'mvn clean install' I get an enforcer
> error:
>
> [WARNING] Rule 0:
> org.semver.enforcer.RequireSemanticVersioningConformance failed with
> message:
>
> Current codebase is incompatible with version <1.5.0>. Version should
> be at least <2.0.0>.
>
> Tom

If I run `mvn clean install` on the release candidate, it succeeds. I 
looked at your error gist [1] and I don't see one of the methods it 
complains about, `mergeKeyValueMetadata`, in 1.5.0 [2]. (The `init` 
method is provided by the ReadSupport, which semver can't catch.)

Can you check your repository's copy of parquet-avro-1.5.0? My sha1 is 
this (which matches the .sha1 file):
   2a44461074b5624a5d8892ce1daf33bc34207e00  .../parquet-avro-1.5.0.jar

Looks like Travis CI is successful as well [3]. Can someone else try this?

rb

[1]: https://gist.github.com/tomwhite/5232b3e929d223618b5e
[2]: 
https://github.com/apache/incubator-parquet-mr/blob/parquet-1.5.0/parquet-avro/src/main/java/parquet/avro/AvroReadSupport.java
[3]: https://travis-ci.org/apache/incubator-parquet-mr/builds/57570537


-- 
Ryan Blue
Software Engineer
Cloudera, Inc.

Re: [VOTE] Release Apache Parquet MR (Incubating) 1.6.0 RC2

Posted by Tom White <to...@cloudera.com>.
I checked signature and checksums, ran RAT to check license headers,
verified NOTICE, LICENSE and DISCLAIMER - all these look good.
However, when I build with 'mvn clean install' I get an enforcer
error:

[WARNING] Rule 0:
org.semver.enforcer.RequireSemanticVersioningConformance failed with
message:

Current codebase is incompatible with version <1.5.0>. Version should
be at least <2.0.0>.

Tom

On Wed, Apr 8, 2015 at 12:57 AM, Ryan Blue <bl...@apache.org> wrote:
> Hi everyone,
>
> I propose the following RC to be released as official Apache Parquet 1.6.0
> release.
>
> This is the first Apache release of Parquet MR. Like the 2.2.0 release of
> Parquet Format, this uses the old parquet.* package names and will be
> immediately followed by a 1.7.0 release that is strictly a move to
> org.apache.parquet.* packages and artifacts.
>
> The commit id is 4f660778f89d4164c72676f6b15bbc74c0d09373.
> * http://s.apache.org/apache-parquet-mr-1.6.0-incubating-rc2
> * https://github.com/apache/incubator-parquet-mr/tree/4f660778
>
> The release tarball, signature, and checksums are here:
> *
> https://dist.apache.org/repos/dist/dev/incubator/parquet/apache-parquet-mr-1.6.0-incubating-rc2/
>
> You can find the KEYS file here:
> * https://dist.apache.org/repos/dist/dev/incubator/parquet/KEYS
>
> Full changes for the release can be found here:
> *
> https://github.com/apache/incubator-parquet-mr/blob/4f660778/CHANGES.md#version-160
>
> Please download, verify, and test.
>
> This vote will close in 3 days, so please vote by Friday evening, PDT.
>
> [ ] +1 Release this as Apache Parquet Format 1.6.0
> [ ] +0
> [ ] -1 Do not release this because...
>
> --
> Ryan Blue

Re: [VOTE] Release Apache Parquet MR (Incubating) 1.6.0 RC2

Posted by Julien Le Dem <ju...@twitter.com.INVALID>.
I forgot.
But yes +1

On Thu, Apr 9, 2015 at 2:14 PM, Julien Le Dem <ju...@twitter.com> wrote:

> I followed the instruction on the wiki:
> https://cwiki.apache.org/confluence/display/PARQUET/How+To+Verify+A+Release
> - Signature looks good
> - README mentions protobuf and thrift that I have already installed. (I
> would suggest to mention this in the email to the IPMC)
> - ran the build: looks good
> - NOTICE and LICENSE files look good
>
>
>
> On Thu, Apr 9, 2015 at 8:57 AM, Daniel Weeks <dw...@netflix.com.invalid>
> wrote:
>
>> +1
>>
>> Compile, test, and read/wrote using our production data.
>>
>> On Thu, Apr 9, 2015 at 7:30 AM, Chris Aniszczyk <ca...@gmail.com>
>> wrote:
>>
>> > +1
>> >
>> > compiles/tests pass (mvn clean install) and all necessary files look to
>> be
>> > there too
>> >
>> > On Wed, Apr 8, 2015 at 10:19 PM, Jake Farrell <jf...@apache.org>
>> wrote:
>> >
>> > > +1
>> > >
>> > > Checksums and signatures match
>> > > License, Notice and Disclaimer look good
>> > > All tests pass and source compiles successfully
>> > >
>> > > -Jake
>> > >
>> > >
>> > > On Tue, Apr 7, 2015 at 7:57 PM, Ryan Blue <bl...@apache.org> wrote:
>> > >
>> > > > Hi everyone,
>> > > >
>> > > > I propose the following RC to be released as official Apache Parquet
>> > > 1.6.0
>> > > > release.
>> > > >
>> > > > This is the first Apache release of Parquet MR. Like the 2.2.0
>> release
>> > of
>> > > > Parquet Format, this uses the old parquet.* package names and will
>> be
>> > > > immediately followed by a 1.7.0 release that is strictly a move to
>> > > > org.apache.parquet.* packages and artifacts.
>> > > >
>> > > > The commit id is 4f660778f89d4164c72676f6b15bbc74c0d09373.
>> > > > * http://s.apache.org/apache-parquet-mr-1.6.0-incubating-rc2
>> > > > * https://github.com/apache/incubator-parquet-mr/tree/4f660778
>> > > >
>> > > > The release tarball, signature, and checksums are here:
>> > > > * https://dist.apache.org/repos/dist/dev/incubator/parquet/
>> > > > apache-parquet-mr-1.6.0-incubating-rc2/
>> > > >
>> > > > You can find the KEYS file here:
>> > > > * https://dist.apache.org/repos/dist/dev/incubator/parquet/KEYS
>> > > >
>> > > > Full changes for the release can be found here:
>> > > > * https://github.com/apache/incubator-parquet-mr/blob/
>> > > > 4f660778/CHANGES.md#version-160
>> > > >
>> > > > Please download, verify, and test.
>> > > >
>> > > > This vote will close in 3 days, so please vote by Friday evening,
>> PDT.
>> > > >
>> > > > [ ] +1 Release this as Apache Parquet Format 1.6.0
>> > > > [ ] +0
>> > > > [ ] -1 Do not release this because...
>> > > >
>> > > > --
>> > > > Ryan Blue
>> > > >
>> > >
>> >
>> >
>> >
>> > --
>> > Cheers,
>> >
>> > Chris Aniszczyk
>> > http://aniszczyk.org
>> > +1 512 961 6719
>> >
>>
>
>

Re: [VOTE] Release Apache Parquet MR (Incubating) 1.6.0 RC2

Posted by Julien Le Dem <ju...@twitter.com.INVALID>.
I followed the instruction on the wiki:
https://cwiki.apache.org/confluence/display/PARQUET/How+To+Verify+A+Release
- Signature looks good
- README mentions protobuf and thrift that I have already installed. (I
would suggest to mention this in the email to the IPMC)
- ran the build: looks good
- NOTICE and LICENSE files look good



On Thu, Apr 9, 2015 at 8:57 AM, Daniel Weeks <dw...@netflix.com.invalid>
wrote:

> +1
>
> Compile, test, and read/wrote using our production data.
>
> On Thu, Apr 9, 2015 at 7:30 AM, Chris Aniszczyk <ca...@gmail.com>
> wrote:
>
> > +1
> >
> > compiles/tests pass (mvn clean install) and all necessary files look to
> be
> > there too
> >
> > On Wed, Apr 8, 2015 at 10:19 PM, Jake Farrell <jf...@apache.org>
> wrote:
> >
> > > +1
> > >
> > > Checksums and signatures match
> > > License, Notice and Disclaimer look good
> > > All tests pass and source compiles successfully
> > >
> > > -Jake
> > >
> > >
> > > On Tue, Apr 7, 2015 at 7:57 PM, Ryan Blue <bl...@apache.org> wrote:
> > >
> > > > Hi everyone,
> > > >
> > > > I propose the following RC to be released as official Apache Parquet
> > > 1.6.0
> > > > release.
> > > >
> > > > This is the first Apache release of Parquet MR. Like the 2.2.0
> release
> > of
> > > > Parquet Format, this uses the old parquet.* package names and will be
> > > > immediately followed by a 1.7.0 release that is strictly a move to
> > > > org.apache.parquet.* packages and artifacts.
> > > >
> > > > The commit id is 4f660778f89d4164c72676f6b15bbc74c0d09373.
> > > > * http://s.apache.org/apache-parquet-mr-1.6.0-incubating-rc2
> > > > * https://github.com/apache/incubator-parquet-mr/tree/4f660778
> > > >
> > > > The release tarball, signature, and checksums are here:
> > > > * https://dist.apache.org/repos/dist/dev/incubator/parquet/
> > > > apache-parquet-mr-1.6.0-incubating-rc2/
> > > >
> > > > You can find the KEYS file here:
> > > > * https://dist.apache.org/repos/dist/dev/incubator/parquet/KEYS
> > > >
> > > > Full changes for the release can be found here:
> > > > * https://github.com/apache/incubator-parquet-mr/blob/
> > > > 4f660778/CHANGES.md#version-160
> > > >
> > > > Please download, verify, and test.
> > > >
> > > > This vote will close in 3 days, so please vote by Friday evening,
> PDT.
> > > >
> > > > [ ] +1 Release this as Apache Parquet Format 1.6.0
> > > > [ ] +0
> > > > [ ] -1 Do not release this because...
> > > >
> > > > --
> > > > Ryan Blue
> > > >
> > >
> >
> >
> >
> > --
> > Cheers,
> >
> > Chris Aniszczyk
> > http://aniszczyk.org
> > +1 512 961 6719
> >
>

Re: [VOTE] Release Apache Parquet MR (Incubating) 1.6.0 RC2

Posted by Daniel Weeks <dw...@netflix.com.INVALID>.
+1

Compile, test, and read/wrote using our production data.

On Thu, Apr 9, 2015 at 7:30 AM, Chris Aniszczyk <ca...@gmail.com>
wrote:

> +1
>
> compiles/tests pass (mvn clean install) and all necessary files look to be
> there too
>
> On Wed, Apr 8, 2015 at 10:19 PM, Jake Farrell <jf...@apache.org> wrote:
>
> > +1
> >
> > Checksums and signatures match
> > License, Notice and Disclaimer look good
> > All tests pass and source compiles successfully
> >
> > -Jake
> >
> >
> > On Tue, Apr 7, 2015 at 7:57 PM, Ryan Blue <bl...@apache.org> wrote:
> >
> > > Hi everyone,
> > >
> > > I propose the following RC to be released as official Apache Parquet
> > 1.6.0
> > > release.
> > >
> > > This is the first Apache release of Parquet MR. Like the 2.2.0 release
> of
> > > Parquet Format, this uses the old parquet.* package names and will be
> > > immediately followed by a 1.7.0 release that is strictly a move to
> > > org.apache.parquet.* packages and artifacts.
> > >
> > > The commit id is 4f660778f89d4164c72676f6b15bbc74c0d09373.
> > > * http://s.apache.org/apache-parquet-mr-1.6.0-incubating-rc2
> > > * https://github.com/apache/incubator-parquet-mr/tree/4f660778
> > >
> > > The release tarball, signature, and checksums are here:
> > > * https://dist.apache.org/repos/dist/dev/incubator/parquet/
> > > apache-parquet-mr-1.6.0-incubating-rc2/
> > >
> > > You can find the KEYS file here:
> > > * https://dist.apache.org/repos/dist/dev/incubator/parquet/KEYS
> > >
> > > Full changes for the release can be found here:
> > > * https://github.com/apache/incubator-parquet-mr/blob/
> > > 4f660778/CHANGES.md#version-160
> > >
> > > Please download, verify, and test.
> > >
> > > This vote will close in 3 days, so please vote by Friday evening, PDT.
> > >
> > > [ ] +1 Release this as Apache Parquet Format 1.6.0
> > > [ ] +0
> > > [ ] -1 Do not release this because...
> > >
> > > --
> > > Ryan Blue
> > >
> >
>
>
>
> --
> Cheers,
>
> Chris Aniszczyk
> http://aniszczyk.org
> +1 512 961 6719
>

Re: [VOTE] Release Apache Parquet MR (Incubating) 1.6.0 RC2

Posted by Chris Aniszczyk <ca...@gmail.com>.
+1

compiles/tests pass (mvn clean install) and all necessary files look to be
there too

On Wed, Apr 8, 2015 at 10:19 PM, Jake Farrell <jf...@apache.org> wrote:

> +1
>
> Checksums and signatures match
> License, Notice and Disclaimer look good
> All tests pass and source compiles successfully
>
> -Jake
>
>
> On Tue, Apr 7, 2015 at 7:57 PM, Ryan Blue <bl...@apache.org> wrote:
>
> > Hi everyone,
> >
> > I propose the following RC to be released as official Apache Parquet
> 1.6.0
> > release.
> >
> > This is the first Apache release of Parquet MR. Like the 2.2.0 release of
> > Parquet Format, this uses the old parquet.* package names and will be
> > immediately followed by a 1.7.0 release that is strictly a move to
> > org.apache.parquet.* packages and artifacts.
> >
> > The commit id is 4f660778f89d4164c72676f6b15bbc74c0d09373.
> > * http://s.apache.org/apache-parquet-mr-1.6.0-incubating-rc2
> > * https://github.com/apache/incubator-parquet-mr/tree/4f660778
> >
> > The release tarball, signature, and checksums are here:
> > * https://dist.apache.org/repos/dist/dev/incubator/parquet/
> > apache-parquet-mr-1.6.0-incubating-rc2/
> >
> > You can find the KEYS file here:
> > * https://dist.apache.org/repos/dist/dev/incubator/parquet/KEYS
> >
> > Full changes for the release can be found here:
> > * https://github.com/apache/incubator-parquet-mr/blob/
> > 4f660778/CHANGES.md#version-160
> >
> > Please download, verify, and test.
> >
> > This vote will close in 3 days, so please vote by Friday evening, PDT.
> >
> > [ ] +1 Release this as Apache Parquet Format 1.6.0
> > [ ] +0
> > [ ] -1 Do not release this because...
> >
> > --
> > Ryan Blue
> >
>



-- 
Cheers,

Chris Aniszczyk
http://aniszczyk.org
+1 512 961 6719

Re: [VOTE] Release Apache Parquet MR (Incubating) 1.6.0 RC2

Posted by Jake Farrell <jf...@apache.org>.
+1

Checksums and signatures match
License, Notice and Disclaimer look good
All tests pass and source compiles successfully

-Jake


On Tue, Apr 7, 2015 at 7:57 PM, Ryan Blue <bl...@apache.org> wrote:

> Hi everyone,
>
> I propose the following RC to be released as official Apache Parquet 1.6.0
> release.
>
> This is the first Apache release of Parquet MR. Like the 2.2.0 release of
> Parquet Format, this uses the old parquet.* package names and will be
> immediately followed by a 1.7.0 release that is strictly a move to
> org.apache.parquet.* packages and artifacts.
>
> The commit id is 4f660778f89d4164c72676f6b15bbc74c0d09373.
> * http://s.apache.org/apache-parquet-mr-1.6.0-incubating-rc2
> * https://github.com/apache/incubator-parquet-mr/tree/4f660778
>
> The release tarball, signature, and checksums are here:
> * https://dist.apache.org/repos/dist/dev/incubator/parquet/
> apache-parquet-mr-1.6.0-incubating-rc2/
>
> You can find the KEYS file here:
> * https://dist.apache.org/repos/dist/dev/incubator/parquet/KEYS
>
> Full changes for the release can be found here:
> * https://github.com/apache/incubator-parquet-mr/blob/
> 4f660778/CHANGES.md#version-160
>
> Please download, verify, and test.
>
> This vote will close in 3 days, so please vote by Friday evening, PDT.
>
> [ ] +1 Release this as Apache Parquet Format 1.6.0
> [ ] +0
> [ ] -1 Do not release this because...
>
> --
> Ryan Blue
>