Skip to content

Upgrade Iceberg from 1.4.3 to 1.5.0#21961

Merged
tdcmeehan merged 1 commit intoprestodb:masterfrom
nastra:iceberg-1.5.0
Mar 15, 2024
Merged

Upgrade Iceberg from 1.4.3 to 1.5.0#21961
tdcmeehan merged 1 commit intoprestodb:masterfrom
nastra:iceberg-1.5.0

Conversation

@nastra
Copy link
Contributor

@nastra nastra commented Feb 19, 2024

Description

This bumps Apache Iceberg from 1.4.3 to 1.5.0

== RELEASE NOTES ==

Iceberg Changes
* Upgrade Iceberg version from 1.4.3 to 1.5.0

Note that this currently points to Iceberg 1.5.0 RC0. Once the official release is out, I'll remove usage of the RC repository

@nastra nastra requested a review from a team as a code owner February 19, 2024 13:03
@nastra nastra requested a review from presto-oss February 19, 2024 13:03
@nastra nastra force-pushed the iceberg-1.5.0 branch 3 times, most recently from 964d93e to a3cc705 Compare February 19, 2024 14:18
@steveburnett
Copy link
Contributor

Nit: following the Order of changes in the release note guidelines, suggest changing the release note entry from Update to Upgrade.

@nastra nastra changed the title Bump Iceberg from 1.4.1 to 1.5.0 Upgrade Iceberg from 1.4.1 to 1.5.0 Feb 19, 2024
@tdcmeehan tdcmeehan self-assigned this Feb 20, 2024
@nastra
Copy link
Contributor Author

nastra commented Feb 22, 2024

There's one unrelated CI failure:

2024-02-21T17:07:51.8918624Z [ERROR] com.facebook.presto.execution.resourceGroups.db.TestDistributedQueuesDb.testResourceGroupConcurrencyThreshold  Time elapsed: 0.048 s  <<< FAILURE!
2024-02-21T17:07:51.8922653Z org.testng.internal.thread.ThreadTimeoutException: Method com.facebook.presto.execution.resourceGroups.db.TestDistributedQueuesDb.testResourceGroupConcurrencyThreshold() didn't finish within the time-out 60000
2024-02-21T17:07:51.8926010Z 	at org.testng.internal.invokers.MethodInvocationHelper.invokeWithTimeoutWithNoExecutor(MethodInvocationHelper.java:347)
2024-02-21T17:07:51.8928169Z 	at org.testng.internal.invokers.MethodInvocationHelper.invokeWithTimeout(MethodInvocationHelper.java:294)
2024-02-21T17:07:51.8929987Z 	at org.testng.internal.invokers.TestInvoker.invokeMethod(TestInvoker.java:679)
2024-02-21T17:07:51.8931662Z 	at org.testng.internal.invokers.TestInvoker.invokeTestMethod(TestInvoker.java:220)
2024-02-21T17:07:51.8933311Z 	at org.testng.internal.invokers.MethodRunner.runInSequence(MethodRunner.java:50)
2024-02-21T17:07:51.8935072Z 	at org.testng.internal.invokers.TestInvoker$MethodInvocationAgent.invoke(TestInvoker.java:945)
2024-02-21T17:07:51.8936754Z 	at org.testng.internal.invokers.TestInvoker.invokeTestMethods(TestInvoker.java:193)
2024-02-21T17:07:51.8938231Z 	at org.testng.internal.invokers.TestMethodWorker.invokeTestMethods(TestMethodWorker.java:146)
2024-02-21T17:07:51.8939661Z 	at org.testng.internal.invokers.TestMethodWorker.run(TestMethodWorker.java:128)
2024-02-21T17:07:51.8940989Z 	at java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1149)
2024-02-21T17:07:51.8942452Z 	at java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:624)
2024-02-21T17:07:51.8943422Z 	at java.lang.Thread.run(Thread.java:750)

@steveburnett
Copy link
Contributor

There's one unrelated CI failure:

I looked at the specific test that had failed, noticed that "rerun failed tests" was enabled for the specific failure, and re-ran it in case it was an intermittent failure.
Now showing "All checks have passed".

@nastra nastra changed the title Upgrade Iceberg from 1.4.1 to 1.5.0 Upgrade Iceberg from 1.4.3 to 1.5.0 Mar 12, 2024
@nastra
Copy link
Contributor Author

nastra commented Mar 12, 2024

This should be ready for review as Iceberg 1.5.0 has been officially released now: https://iceberg.apache.org/releases/#150-release

@tdcmeehan tdcmeehan merged commit ee2e5c4 into prestodb:master Mar 15, 2024
@nastra nastra deleted the iceberg-1.5.0 branch March 16, 2024 09:28
@wanglinsong wanglinsong mentioned this pull request May 1, 2024
48 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants