Skip to content

Nessie: Upgrade nessie dependencies to 0.50.0#6875

Merged
Fokko merged 1 commit into
apache:masterfrom
dimas-b:nessie-upgrade
Mar 6, 2023
Merged

Nessie: Upgrade nessie dependencies to 0.50.0#6875
Fokko merged 1 commit into
apache:masterfrom
dimas-b:nessie-upgrade

Conversation

@dimas-b
Copy link
Copy Markdown
Contributor

@dimas-b dimas-b commented Feb 17, 2023

@github-actions github-actions Bot added the build label Feb 17, 2023
@dimas-b
Copy link
Copy Markdown
Contributor Author

dimas-b commented Feb 17, 2023

cc: @snazy @ajantha-bhat

@dimas-b dimas-b marked this pull request as draft February 17, 2023 22:14
@dimas-b
Copy link
Copy Markdown
Contributor Author

dimas-b commented Feb 17, 2023

Hmmm... TestNessieTable passed locally, but failed in CI 🤔 I'll investigate next week.

@dimas-b
Copy link
Copy Markdown
Contributor Author

dimas-b commented Feb 21, 2023

CI is broken because of projectnessie/nessie#6156

@dimas-b dimas-b changed the title Nessie: Upgrade projectnessie dependencies to 0.49.0 Nessie: Upgrade nessie dependencies to 0.50.0 Feb 24, 2023
Copy link
Copy Markdown
Member

@ajantha-bhat ajantha-bhat left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM.

nit: we can add release notes link in the PR description.

@dimas-b dimas-b marked this pull request as ready for review February 24, 2023 16:07
@snazy
Copy link
Copy Markdown
Member

snazy commented Mar 2, 2023

@Fokko could you merge this one?

@dimas-b
Copy link
Copy Markdown
Contributor Author

dimas-b commented Mar 6, 2023

Rebased (for a CI run on the latest codebase)

Copy link
Copy Markdown
Contributor

@Fokko Fokko left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Will merge after a green CI

@Fokko Fokko merged commit 50e9d01 into apache:master Mar 6, 2023
@Fokko
Copy link
Copy Markdown
Contributor

Fokko commented Mar 6, 2023

Thanks @dimas-b for the PR, and fixing the issue in 0.49. @ajantha-bhat and @snazy for the review!

@dimas-b
Copy link
Copy Markdown
Contributor Author

dimas-b commented Mar 6, 2023

Thanks, @Fokko !

@dimas-b dimas-b deleted the nessie-upgrade branch March 6, 2023 20:32
krvikash pushed a commit to krvikash/iceberg that referenced this pull request Mar 16, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants