Ran all steps successfully. +1 from me.
On Mon, Oct 14, 2019 at 7:30 AM 俊杰陈 <cjjnj...@gmail.com> wrote: > Ran all steps successfully, +1 > > On Mon, Oct 14, 2019 at 7:39 AM Ted Gooch <tgo...@netflix.com.invalid> > wrote: > > > > Ran all steps no issues from me. > > +1 > > > > On Sun, Oct 13, 2019 at 12:09 PM Ryan Blue <rb...@netflix.com.invalid> > wrote: > >> > >> +1 (binding) > >> > >> I went through all of the validation and it looks good. > >> > >> I also tested the iceberg-spark-runtime Jar with the Apache Spark 2.4.4 > download. Copying the runtime Jar into Spark's jars folder works without > problems to read and write both path-based tables and Hive tables. Metadata > tables work correctly, same with time travel, and metadata tables with time > travel also work. I also didn't run out of threads in the test Hive > metastore as I did with the last candidate. > >> > >> On Sun, Oct 13, 2019 at 11:30 AM Anton Okolnychyi < > aokolnyc...@apple.com> wrote: > >>> > >>> +1 from me then > >>> > >>> On 13 Oct 2019, at 18:33, Ryan Blue <rb...@netflix.com.INVALID> wrote: > >>> > >>> The publish steps will now sign all of the artifacts, which is > required for an Apache release. That's why the publish steps fail in > master. To fix this in master, we can come up with a way to only turn on > release signatures if `-Prelease` is set, which is how we also select the > Apache releases repository. > >>> > >>> I don't think this is a problem with the release. The convenience > binaries in the release must be signed and published from an Apache > repository, so this is necessary. If you're trying to use the release, then > you don't need to be using JitPack. > >>> > >>> On Sun, Oct 13, 2019 at 6:53 AM Anton Okolnychyi > <aokolnyc...@apple.com.invalid> wrote: > >>>> > >>>> Verified signature/checksum/rat, run tests. > >>>> > >>>> No other pending questions except what Arina and Gautam brought up. > >>>> > >>>> - Anton > >>>> > >>>> On 13 Oct 2019, at 09:17, Gautam <gautamkows...@gmail.com> wrote: > >>>> > >>>> I was able to run steps in Ryan's mail just fine but ran into the > same thing Arina mentioned .. when running " ./graldew build publish " .. > >>>> > >>>> A problem was found with the configuration of task > ':iceberg-api:signApachePublication'. > >>>> > No value has been specified for property 'signatory.keyId'. > >>>> > >>>> > >>>> Something we are expected to do with the keys here? > >>>> > >>>> > >>>> > >>>> > >>>> > >>>> > >>>> On Sat, Oct 12, 2019 at 8:30 PM Arina Yelchiyeva < > arina.yelchiy...@gmail.com> wrote: > >>>>> > >>>>> Not sure, if this is related to the release vote but after "Update > build for Apache releases" commit [1], we are not longer able to build > Iceberg using JitPack. > >>>>> Error [2]: > >>>>> > >>>>> * What went wrong: > >>>>> A problem was found with the configuration of task > ':iceberg-api:signApachePublication'. > >>>>> > No value has been specified for property 'signatory.keyId'. > >>>>> > >>>>> > >>>>> [1] > https://github.com/apache/incubator-iceberg/commit/2219c86ec6dc5512b2e581f500125841b1b56226 > >>>>> [2] > https://jitpack.io/com/github/apache/incubator-iceberg/5620f119f4/build.log > >>>>> > >>>>> On Oct 12, 2019, at 6:08 AM, Julien Le Dem <julien.le...@gmail.com> > wrote: > >>>>> > >>>>> I’m away for a long weekend without my laptop and will be able to > try it out on Tuesday (if votes are still needed). > >>>>> Happy validation everyone! > >>>>> Julien > >>>>> > >>>>> On Oct 11, 2019, at 18:21, Ryan Blue <b...@apache.org> wrote: > >>>>> > >>>>> Here are the steps I included on the last thread if you'd like to > validate the release: > >>>>> > >>>>> Download the source tarball, signature (.asc), and checksum (.sha512) > >>>>> Import gpg keys: download KEYS and run gpg --import > /path/to/downloaded/KEYS (optional if this hasn’t changed) > >>>>> Verify the signature by running: gpg --verify > apache-iceberg-0.7.0-incubating.tar.gz.asc > >>>>> Verify the checksum by running: sha512sum -c > apache-iceberg-0.7.0-incubating.tar.gz.sha512 > >>>>> Untar the archive and go into the source directory: tar xzf > apache-iceberg-0.7.0-incubating.tar.gz && cd apache-iceberg-0.7.0-incubating > >>>>> Run RAT checks to validate license headers: dev/check-license > >>>>> Build and test the project: ./gradlew build (use Java 8) > >>>>> > >>>>> You can also validate the LICENSE and NOTICE documentation, which is > included in the source tarball, as well as the staged binary artifacts. > >>>>> > >>>>> To validate the convenience binaries, add the Maven URL from the > email above to a downstream project and update your Iceberg dependency to > 0.7.0-incubating, like this: > >>>>> > >>>>> repositories { > >>>>> maven { > >>>>> name 'stagedIceberg' > >>>>> url ' > https://repository.apache.org/content/repositories/orgapacheiceberg-1000/' > >>>>> } > >>>>> } > >>>>> > >>>>> ext { > >>>>> icebergVersion = '0.7.0-incubating' > >>>>> } > >>>>> > >>>>> Then run the downstream project’s tests. > >>>>> > >>>>> Thanks for voting, everyone! > >>>>> > >>>>> rb > >>>>> > >>>>> > >>>>> On Fri, Oct 11, 2019 at 6:18 PM Ryan Blue <b...@apache.org> wrote: > >>>>>> > >>>>>> Hi everyone, > >>>>>> > >>>>>> I propose the following RC to be released as official Apache > Iceberg 0.7.0-incubating release. > >>>>>> > >>>>>> The commit id is 028a8d0e65d9c713b9b040c592fa10641b6c867b > >>>>>> * This corresponds to the tag: apache-iceberg-0.7.0-incubating-rc1 > >>>>>> * > https://github.com/apache/incubator-iceberg/tree/apache-iceberg-0.7.0-incubating-rc1 > >>>>>> * > https://github.com/apache/incubator-iceberg/tree/028a8d0e65d9c713b9b040c592fa10641b6c867b > >>>>>> > >>>>>> The release tarball, signature, and checksums are here: > >>>>>> * > https://dist.apache.org/repos/dist/dev/incubator/iceberg/apache-iceberg-0.7.0-incubating-rc1/ > >>>>>> > >>>>>> You can find the KEYS file here: > >>>>>> * https://dist.apache.org/repos/dist/dev/incubator/iceberg/KEYS > >>>>>> > >>>>>> Convenience binary artifacts are staged in Nexus. The Maven > repository URL is: > >>>>>> * > https://repository.apache.org/content/repositories/orgapacheiceberg-1001/ > >>>>>> > >>>>>> This is the first Apache Iceberg release. > >>>>>> > >>>>>> Please download, verify, and test; then vote in the next 72 hours. > >>>>>> > >>>>>> [ ] +1 Release this as Apache Parquet 0.7.0-incubating > >>>>>> [ ] +0 > >>>>>> [ ] -1 Do not release this because... > >>>>>> > >>>>>> -- > >>>>>> Ryan Blue > >>>>> > >>>>> > >>>>> > >>>>> -- > >>>>> Ryan Blue > >>>>> > >>>>> > >>>> > >>> > >>> > >>> -- > >>> Ryan Blue > >>> Software Engineer > >>> Netflix > >>> > >>> > >> > >> > >> -- > >> Ryan Blue > >> Software Engineer > >> Netflix > > > > -- > Thanks & Best Regards >