apache / kyuubi

Apache Kyuubi is a distributed and multi-tenant gateway to provide serverless SQL on data warehouses and lakehouses.
https://kyuubi.apache.org/
Apache License 2.0
2.11k stars 914 forks source link

Bump Iceberg from 1.6.1 to 1.7.0 #6804

Closed pionCham closed 1 week ago

pionCham commented 1 week ago

:mag: Description

Issue References ๐Ÿ”—

Apache Iceberg 1.7.0 release https://github.com/apache/iceberg/releases/tag/apache-iceberg-1.7.0

Describe Your Solution ๐Ÿ”ง

Types of changes :bookmark:

Test Plan ๐Ÿงช

Behavior Without This Pull Request :coffin:

Behavior With This Pull Request :tada:

Related Unit Tests


Checklist ๐Ÿ“

Be nice. Be informative.

bowenliang123 commented 1 week ago

Iceberg 1.7.0 has dropped the support for Java 8. Let's see whether it's passing the CI tests.

pan3793 commented 1 week ago

Oh, it drops Java 8 ...

bowenliang123 commented 1 week ago

Oops ... it failed. How about bumping Iceberg to 1.7.0 for all except for the java-8 profile ?

codecov-commenter commented 1 week ago

Codecov Report

All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 0.00%. Comparing base (dddb037) to head (0896ac7). Report is 4 commits behind head on master.

Additional details and impacted files ```diff @@ Coverage Diff @@ ## master #6804 +/- ## ====================================== Coverage 0.00% 0.00% ====================================== Files 687 687 Lines 42442 42442 Branches 5793 5793 ====================================== Misses 42442 42442 ```

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

pionCham commented 1 week ago

Oops ... it failed. How about bumping Iceberg to 1.7.0 for all except for the java-8 profile ? It sounded good, I tried it.

pionCham commented 1 week ago

LGTM if tests pass, thanks for your contribution!

Tests look like they've passed .

pan3793 commented 1 week ago

lgtm if CI pass