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

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

Copy link
Copy Markdown
Contributor

@github-actions github-actions Bot added the build label Feb 17, 2023
@dimas-b

dimas-b commented Feb 17, 2023

Copy link
Copy Markdown
Contributor Author

cc: @snazy @ajantha-bhat

@dimas-b dimas-b marked this pull request as draft February 17, 2023 22:14
@dimas-b

dimas-b commented Feb 17, 2023

Copy link
Copy Markdown
Contributor Author

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

@dimas-b

dimas-b commented Feb 21, 2023

Copy link
Copy Markdown
Contributor Author

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

@ajantha-bhat ajantha-bhat left a comment

Copy link
Copy Markdown
Member

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

snazy commented Mar 2, 2023

Copy link
Copy Markdown
Member

@Fokko could you merge this one?

@dimas-b

dimas-b commented Mar 6, 2023

Copy link
Copy Markdown
Contributor Author

Rebased (for a CI run on the latest codebase)

@Fokko Fokko left a comment

Copy link
Copy Markdown
Contributor

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

Fokko commented Mar 6, 2023

Copy link
Copy Markdown
Contributor

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

@dimas-b

dimas-b commented Mar 6, 2023

Copy link
Copy Markdown
Contributor Author

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