Build: Revert Parquet to 1.13.1

[RussellSpitzer]

apache/parquet-java#3040 -
Was discovered by @pan3793 in Parquet 1.14.(0,1,2,3). This is considered a critical data loss issue so we will pull back parquet to avoid the vulnerability.

[RussellSpitzer]

Some Flink Tests Broken, Rerunning

[RussellSpitzer]

Same test failed again,

> Task :iceberg-flink:iceberg-flink-1.18:test

TestMetadataTableReadableMetrics > testPrimitiveColumns() > catalogName=testhive, baseNamespace= FAILED
    java.lang.AssertionError: 
    Expecting actual:
      [+I[+I[+I[52, 4, 2, null, [-41, 93, 117], [-37, 109, -74]], +I[32, 4, 0, null, false, true], +I[85, 4, 1, null, 1.00, 2.00], +I[85, 4, 0, 1, 1.0, 2.0], +I[44, 4, 2, null, [-41, 93, 117], [-37, 109, -74]], +I[71, 4, 0, 2, 0.0, 0.0], +I[71, 4, 0, null, 1, 2], +I[79, 4, 0, null, 1, 2], +I[79, 4, 0, null, 1, 2]]]]
    to contain exactly in any order:
      [+I[+I[+I[55, 4, 2, null, [-41, 93, 117], [-37, 109, -74]], +I[36, 4, 0, null, false, true], +I[91, 4, 1, null, 1.00, 2.00], +I[91, 4, 0, 1, 1.0, 2.0], +I[47, 4, 2, null, [-41, 93, 117], [-37, 109, -74]], +I[77, 4, 0, 2, 0.0, 0.0], +I[77, 4, 0, null, 1, 2], +I[85, 4, 0, null, 1, 2], +I[85, 4, 0, null, 1, 2]]]]
    elements not found:
      [+I[+I[+I[55, 4, 2, null, [-41, 93, 117], [-37, 109, -74]], +I[36, 4, 0, null, false, true], +I[91, 4, 1, null, 1.00, 2.00], +I[91, 4, 0, 1, 1.0, 2.0], +I[47, 4, 2, null, [-41, 93, 117], [-37, 109, -74]], +I[77, 4, 0, 2, 0.0, 0.0], +I[77, 4, 0, null, 1, 2], +I[85, 4, 0, null, 1, 2], +I[85, 4, 0, null, 1, 2]]]]
    and elements not expected:
      [+I[+I[+I[52, 4, 2, null, [-41, 93, 117], [-37, 109, -74]], +I[32, 4, 0, null, false, true], +I[85, 4, 1, null, 1.00, 2.00], +I[85, 4, 0, 1, 1.0, 2.0], +I[44, 4, 2, null, [-41, 93, 117], [-37, 109, -74]], +I[71, 4, 0, 2, 0.0, 0.0], +I[71, 4, 0, null, 1, 2], +I[79, 4, 0, null, 1, 2], +I[79, 4, 0, null, 1, 2]]]]
        at org.apache.iceberg.flink.TestHelpers.assertRows(TestHelpers.java:162)
        at org.apache.iceberg.flink.source.TestMetadataTableReadableMetrics.testPrimitiveColumns(TestMetadataTableReadableMetrics.java:255)
        ```

[RussellSpitzer]

Also fails locally

[Fokko]

@RussellSpitzer Easiest is probably to revert this commit: b8c2b20

[RussellSpitzer]

Thanks @Fokko ! Note that we should probably fix these tests so that they don't hardwire expected values irrelevant fields

[RussellSpitzer]

Please review the 1.7.0 RC1 CherryPicks #11466

…

On Mon, Nov 4, 2024 at 12:18 PM Fokko Driesprong ***@***.***> wrote: @RussellSpitzer <https://github.com/RussellSpitzer> Easiest is probably to revert this commit: b8c2b20 <b8c2b20> — Reply to this email directly, view it on GitHub <#11462 (comment)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/AADE2YNQIAS2UTQ66S65AMLZ66263AVCNFSM6AAAAABREWUJQKVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDINJVGQYDCMRRHE> . You are receiving this because you were mentioned.Message ID: ***@***.***>