Infra, Docs: Publish Apache Iceberg 1.6.0 release #10752
Conversation
|
@Fokko do you mind to take a look on this one ? |
|
@nastra, @jbonofre: Since release notes gonna take sometime, I have opened #10754 because I think open PR is failing (#10753 (comment)) because we have a new tag and old version is not matching? |
Fokko
left a comment
Fokko
left a comment
There was a problem hiding this comment.
Looks good, thanks @jbonofre for working on this 👍
@ajantha-bhat Wanted to check if you have any more comments before merging this
nastra
left a comment
nastra
left a comment
There was a problem hiding this comment.
RevAPI update is still missing, see also my earlier comment: #10752 (comment)
|
@nastra revapi has been updated already by another PR, so not included in this one. |
ah sorry I missed that update |
nastra
left a comment
nastra
left a comment
There was a problem hiding this comment.
LGTM with a few minor comments
amogh-jahagirdar
changed the title
There was a problem hiding this comment.
Thanks @jbonofre! Thanks @ajantha-bhat @Fokko @nastra @hussein-awala for reviewing.
From my side looks good, seems like we wanted to get in dependency updates in the release notes based on https://github.com/apache/iceberg/pull/10752/files#r1689845177? So I'll hold off on merging
Co-authored-by: Eduard Tudenhoefner <[email protected]>
|
@amogh-jahagirdar as requested by @nastra I added a |
|
Thanks @jbonofre ! |
|
@amogh-jahagirdar thanks ! I will do the announcement. |
Co-authored-by: Eduard Tudenhoefner <[email protected]>
6 participants
No description provided.