Thanks for running the release Gabor!

The signature checks out:
MacBook-Pro-van-Fokko:Downloads fokkodriesprong$ curl
https://downloads.apache.org/parquet/KEYS > KEYS
  % Total    % Received % Xferd  Average Speed   Time    Time     Time
 Current
                                 Dload  Upload   Total   Spent    Left
 Speed
100 33082  100 33082    0     0   212k      0 --:--:-- --:--:-- --:--:--
 212k
MacBook-Pro-van-Fokko:Downloads fokkodriesprong$ gpg --import  KEYS
gpg: key 97D7E8647AE7E47B: 2 signatures not checked due to missing keys
gpg: key 97D7E8647AE7E47B: public key "Julien Le Dem <[email protected]>"
imported
gpg: key 7CD8278971F0F13B: 1 signature not checked due to a missing key
gpg: key 7CD8278971F0F13B: public key "Tianshuo Deng <[email protected]>"
imported
gpg: key 4FB955854318F669: 8 signatures not checked due to missing keys
gpg: key 4FB955854318F669: public key "Tom White (CODE SIGNING KEY) <
[email protected]>" imported
gpg: key FCB3CBD9D3924CCD: public key "Ryan Blue (CODE SIGNING KEY) <
[email protected]>" imported
gpg: key A9358ED82F7D7992: public key "Alex Levenson <
[email protected]>" imported
gpg: key 442C7FAC7C58F0FE: public key "Alex Levenson <[email protected]>"
imported
gpg: key 29D94E228CAAD602: 2 signatures not checked due to missing keys
gpg: key 29D94E228CAAD602: public key "Uwe L. Korn <[email protected]>"
imported
gpg: key 021057DBF048F543: public key "Gabor Szadovszky <[email protected]>"
imported
gpg: Total number processed: 8
gpg:               imported: 8
gpg: marginals needed: 3  completes needed: 1  trust model: pgp
gpg: depth: 0  valid:   1  signed:   0  trust: 0-, 0q, 0n, 0m, 0f, 1u
gpg: next trustdb check due at 2021-04-08
MacBook-Pro-van-Fokko:Downloads fokkodriesprong$ gpg --verify
apache-parquet-1.12.0.tar.gz.asc apache-parquet-1.12.0.tar.gz
gpg: Signature made wo 27 jan 16:36:01 2021 CET
gpg:                using RSA key 6FB82970311551C7CEF131F5021057DBF048F543
gpg: Good signature from "Gabor Szadovszky <[email protected]>" [unknown]
gpg: WARNING: This key is not certified with a trusted signature!
gpg:          There is no indication that the signature belongs to the
owner.
Primary key fingerprint: 6FB8 2970 3115 51C7 CEF1  31F5 0210 57DB F048 F543

Also, the hash is looking good:
MacBook-Pro-van-Fokko:Downloads fokkodriesprong$ shasum -a 512
apache-parquet-1.12.0.tar.gz
f19e13e3997027f66f82a8cbacff8e832545250a1d84292234d3da45effdcf41a42288dad37ee8f4d04e29415bc58e2f170abdacd9c3ffe6bff4f68d117c32ec
 apache-parquet-1.12.0.tar.gz
MacBook-Pro-van-Fokko:Downloads fokkodriesprong$ cat
apache-parquet-1.12.0.tar.gz.sha512
f19e13e3997027f66f82a8cbacff8e832545250a1d84292234d3da45effdcf41a42288dad37ee8f4d04e29415bc58e2f170abdacd9c3ffe6bff4f68d117c32ec
 apache-parquet-1.12.0.tar.gz

I've checked against Iceberg, and the API is broken for the OutputFile:

MacBook-Pro-van-Fokko:incubator-iceberg fokkodriesprong$ git diff

*diff --git
a/parquet/src/main/java/org/apache/iceberg/parquet/ParquetIO.java
b/parquet/src/main/java/org/apache/iceberg/parquet/ParquetIO.java*

*index d65b8d63..8ce32a7b 100644*

*--- a/parquet/src/main/java/org/apache/iceberg/parquet/ParquetIO.java*

*+++ b/parquet/src/main/java/org/apache/iceberg/parquet/ParquetIO.java*

@@ -162,6 +162,11 @@ class ParquetIO {

     public long defaultBlockSize() {

       return 0;

     }

+

+    @Override

+    public String getPath() {

+      return this.file.location();

+    }

   }

This change is introduced here:
https://github.com/apache/parquet-mr/commit/5c6916c23cb2b9c225ea80328550ee0e11aee225

It is breaking, but not sure if it is blocking.

A +1 (non-binding) from my side!

Cheers, Fokko




Op wo 27 jan. 2021 om 16:46 schreef Gabor Szadovszky <[email protected]>:

> Hi everyone,
>
> I propose the following RC to be released as the official Apache Parquet
> 1.12.0 release.
>
> The commit id is ad59c33e53276572c105b4ccac71293e988adc30
> * This corresponds to the tag: apache-parquet-1.12.0-rc1
> *
>
> https://github.com/apache/parquet-mr/tree/ad59c33e53276572c105b4ccac71293e988adc30
>
> The release tarball, signature, and checksums are here:
> * https://dist.apache.org/repos/dist/dev/parquet/apache-parquet-1.12.0-rc1
>
> You can find the KEYS file here:
> * https://downloads.apache.org/parquet/KEYS
>
> Binary artifacts are staged in Nexus here:
> * https://repository.apache.org/content/groups/staging/org/apache/parquet/
>
> This release includes the features Parquet Modular Encryption and Parquet
> Bloom Filter. See details at:
> *
>
> https://github.com/apache/parquet-mr/blob/apache-parquet-1.12.0-rc1/CHANGES.md
>
> Please download, verify, and test.
>
> Please vote in the next 72 hours.
>
> [ ] +1 Release this as Apache Parquet 1.12.0
> [ ] +0
> [ ] -1 Do not release this because...
>
>
> PS.: Starting with RC1 instead of RC0 because I've missed to update the
> CHANGES.md at the first time.
>

Reply via email to