+1 (binding) * Verified signature and checksum * Build and tested on OSX using `mvn clean install -Dthrift.executable=/usr/local/opt/[email protected]/bin/thrift`
On Fri, Apr 6, 2018, at 6:25 PM, Ryan Blue wrote: > Make that -D, not -P. > > On Fri, Apr 6, 2018 at 9:24 AM, Ryan Blue <[email protected]> wrote: > > > You can either put 0.9 earlier in your PATH, or set thrift.executable: > > > > mvn clean install -Pthrift.executable=/path/to/bin/thrift > > > > > > > > On Fri, Apr 6, 2018 at 9:11 AM, Uwe L. Korn <[email protected]> wrote: > > > >> The build is failing for me because it is picking up my installation of > >> Thrift 0.11. Is there a variable I could set to point it to my Thrift 0.9 > >> installation? > >> > >> Uwe > >> > >> On Fri, Apr 6, 2018, at 2:34 PM, Zoltan Ivanfi wrote: > >> > I would have preferred waiting for the parquet-format release (which > >> > unfortunately failed the vote due to lack of interest) before making an > >> RC > >> > for parquet-mr so that it would refer to the latest format. On the other > >> > hand, all relevant updates of parquet-format are in the documentation > >> only, > >> > so it's not a reason for redoing the RC in itself. > >> > > >> > +1 (non-binding) > >> > > >> > - Reviewed commits that were added during the release process > >> without a > >> > proper review. > >> > - Validated checksums and signature. > >> > - Compared the source tarball to the git repo's state at the release > >> > label. > >> > > >> > Zoltan > >> > > >> > On Fri, Apr 6, 2018 at 8:23 AM Gabor Szadovszky < > >> > [email protected]> wrote: > >> > > >> > > +1 (non-binding) > >> > > > >> > > Validated signature, checksums, matched source tarballs with the git > >> repo. > >> > > > >> > > Gabor > >> > > > >> > > > On 6 Apr 2018, at 03:05, Ryan Blue <[email protected]> wrote: > >> > > > > >> > > > And if anyone wants to try out the new command-line interface, you > >> can > >> > > call > >> > > > it like this: > >> > > > > >> > > > hadoop jar parquet-cli-1.10.0-runtime.jar > >> org.apache.parquet.cli.Main > >> > > > > >> > > > rb > >> > > > > >> > > > > >> > > > On Thu, Apr 5, 2018 at 5:58 PM, Ryan Blue <[email protected]> wrote: > >> > > > > >> > > >> +1 (binding) > >> > > >> > >> > > >> Built, tested, validated signature and checksums, and tested the > >> Iceberg > >> > > >> build with the artifacts. > >> > > >> > >> > > >> On Thu, Apr 5, 2018 at 2:15 PM, Ryan Blue <[email protected]> wrote: > >> > > >> > >> > > >>> Hi everyone, > >> > > >>> > >> > > >>> I propose the following RC to be released as official Apache > >> Parquet > >> > > Java > >> > > >>> 1.10.0 release. > >> > > >>> > >> > > >>> The commit id is 031a6654009e3b82020012a18434c582bd74c73a > >> > > >>> > >> > > >>> - This corresponds to the tag: apache-parquet-1.10.0 > >> > > >>> - https://github.com/apache/parquet-mr/tree/031a665 > >> > > >>> > >> > > >>> The release tarball, signature, and checksums are here: > >> > > >>> > >> > > >>> - https://dist.apache.org/repos/dist/dev/parquet/apache-parque > >> > > >>> t-1.10.0-rc0/ > >> > > >>> > >> > > >>> You can find the KEYS file here: > >> > > >>> > >> > > >>> - https://dist.apache.org/repos/dist/dev/parquet/KEYS > >> > > >>> > >> > > >>> Binary artifacts are staged in Nexus here: > >> > > >>> > >> > > >>> - https://repository.apache.org/content/groups/staging/org/apa > >> > > >>> che/parquet/parquet/1.10.0/ > >> > > >>> > >> > > >>> This release includes: > >> > > >>> > >> > > >>> - The new Parquet command-line tool > >> > > >>> - New APIs to avoid leaking Hadoop classes > >> > > >>> - Fixed sort order for logical types > >> > > >>> - Fixed stats handling for NaN and other floating point edge > >> cases > >> > > >>> > >> > > >>> The full change log is available here: > >> > > >>> > >> > > >>> - https://github.com/apache/parquet-mr/blob/031a665/CHANGES.md > >> > > >>> > >> > > >>> Please download, verify, and test. > >> > > >>> > >> > > >>> Please vote by Tuesday, 10 April 2018. > >> > > >>> > >> > > >>> [ ] +1 Release this as Apache Parquet Java 1.10.0 > >> > > >>> [ ] +0 > >> > > >>> [ ] -1 Do not release this because… > >> > > >>> > >> > > >>> -- > >> > > >>> Ryan Blue > >> > > >>> > >> > > >> > >> > > >> > >> > > >> > >> > > >> -- > >> > > >> Ryan Blue > >> > > >> > >> > > > > >> > > > > >> > > > > >> > > > -- > >> > > > Ryan Blue > >> > > > >> > > > >> > > > >> > > > > > > > > -- > > Ryan Blue > > Software Engineer > > Netflix > > > > > > -- > Ryan Blue > Software Engineer > Netflix
