I agree that we should have a new RC. All files that can have license headers should have them.
On Mon, Oct 16, 2017 at 10:16 AM, Lars Volker <l...@cloudera.com> wrote: > I think it'd be good to create a new RC, but I don't feel strongly about it > and my vote is non-binding. Maybe someone with more experience in the > strictness that's expected from ASF projects can weigh in. > > On Mon, Oct 16, 2017 at 10:08 AM, Wes McKinney <wesmck...@gmail.com> > wrote: > > > Thanks Lars for catching that. I also created about PARQUET-1140 so we > > can be more vigilant about RAT issues. > > > > Do we need an RC1? > > > > On Mon, Oct 16, 2017 at 12:54 PM, Lars Volker <l...@cloudera.com> wrote: > > > 0 (non-binding) > > > > > > * Verified the sha512 sum > > > * Verified the .asc and that it matches Uwe's key > > > * Built and ran the unittests manually on macOS Sierra > > > * Ran the rat tool over the tarball. All warnings looked expected to me > > > except for apache-parquet-cpp-1.3.1/cmake_modules/parquet- > > cppConfig.cmake.in, > > > which I think needs a license header. > > > > > > Would +1 after a license has been added to that file. I created > > > https://github.com/apache/parquet-cpp/pull/411 to fix this. > > > > > > Thank you Uwe for preparing the release! > > > > > > On Mon, Oct 16, 2017 at 12:47 AM, Uwe L. Korn <uw...@xhochy.com> > wrote: > > > > > >> +1 > > >> > > >> * Ran verify-release-candidate on Ubuntu 16.04 > > >> * Ran verify-release-candidate on macOS Sierra > > >> > > >> -- > > >> Uwe L. Korn > > >> uw...@xhochy.com > > >> > > >> On Mon, Oct 16, 2017, at 02:16 AM, Wes McKinney wrote: > > >> > +1 > > >> > > > >> > * Ran verify-release-candidate on Ubuntu 14.04 > > >> > > > >> > In trying to verify the release candidate on MSVC / Visual Studio > > >> > 2015, I found that arrow-reader-writer-test.cc does not compile > > >> > against Arrow 0.7.1. The version in ThirdpartyToolchain.cmake is > > >> > post-0.7.1 > > >> > > > >> > I posted this fix, which also tests 0.7.1 on all the platforms: > > >> > https://github.com/apache/parquet-cpp/pull/410. So I don't think > it's > > >> > necessary to cancel the RC over this > > >> > > > >> > Thanks > > >> > Wes > > >> > > > >> > On Fri, Oct 13, 2017 at 8:24 AM, Uwe L. Korn <uw...@xhochy.com> > > wrote: > > >> > > All, > > >> > > > > >> > > I propose that we accept the following release candidate as the > > >> official > > >> > > Apache Parquet C++ 1.3.1 release. > > >> > > > > >> > > Parquet C++ 1.3.1-rc0 includes the following: > > >> > > --- > > >> > > The CHANGELOG for the release is available at: > > >> > > https://git-wip-us.apache.org/repos/asf?p=parquet-cpp.git&f= > > >> CHANGELOG&hb=apache-parquet-cpp-1.3.1-rc0 > > >> > > > > >> > > The tag used to create the release candidate is: > > >> > > https://git-wip-us.apache.org/repos/asf?p=parquet-cpp.git;a= > > >> shortlog;h=refs/tags/apache-parquet-cpp-1.3.1-rc0 > > >> > > > > >> > > The release candidate is available at: > > >> > > https://dist.apache.org/repos/dist/dev/parquet/apache-parque > > >> t-cpp-1.3.1-rc0/apache-parquet-cpp-1.3.1.tar.gz > > >> > > > > >> > > The MD5 checksum of the release candidate can be found at: > > >> > > https://dist.apache.org/repos/dist/dev/parquet/apache-parque > > >> t-cpp-1.3.1-rc0/apache-parquet-cpp-1.3.1.tar.gz.md5 > > >> > > > > >> > > The signature of the release candidate can be found at: > > >> > > https://dist.apache.org/repos/dist/dev/parquet/apache-parque > > >> t-cpp-1.3.1-rc0/apache-parquet-cpp-1.3.1.tar.gz.asc > > >> > > > > >> > > The GPG key used to sign the release are available at: > > >> > > https://dist.apache.org/repos/dist/dev/parquet/KEYS > > >> > > > > >> > > The release is based on the commit hash > > >> > > a1c950d889a22b267ecddaa3436d3494fcca3ae7. > > >> > > > > >> > > Please download, verify, and test. > > >> > > > > >> > > The vote will close on Mo 16. Okt 15:06:05 CEST 2017 > > >> > > > > >> > > [ ] +1 Release this as Apache Parquet C++ 1.3.1 > > >> > > [ ] +0 > > >> > > [ ] -1 Do not release this as Apache Parquet C++ 1.3.1 because... > > >> > > > -- Ryan Blue Software Engineer Netflix