Skip to content

[MINOR-UPDATE] Upgrade to msgpack 0.9 - #3078

Merged
cgivre merged 1 commit into
apache:masterfrom
apupier:3077-upgrademsgpack
Sep 8, 2026
Merged

[MINOR-UPDATE] Upgrade to msgpack 0.9#3078
cgivre merged 1 commit into
apache:masterfrom
apupier:3077-upgrademsgpack

Conversation

@apupier

@apupier apupier commented Sep 7, 2026

Copy link
Copy Markdown
Contributor

MINOR UDPATE Upgrade to msgpack 0.9

Description

Upgrading from 0.6.6 released in 2012 and having a transitive dependency to junit 4.10 at runtime to 0.9.12 released this year without this junit dependency at runtime

Documentation

N/A

Testing

compilation passed locally of the whole project
common module test are passing

fixes apache#3077

signed-off-by: Aurélien Pupier <apupier@ibm.com>
@cgivre
cgivre marked this pull request as ready for review September 7, 2026 19:22
@cgivre
cgivre marked this pull request as draft September 7, 2026 19:22
@cgivre cgivre changed the title Upgrade to msgpack 0.9 [MINOR-UPDATE] Upgrade to msgpack 0.9 Sep 7, 2026

@cgivre cgivre 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.

LGTM +1
Thank you @apupier

@cgivre

cgivre commented Sep 7, 2026

Copy link
Copy Markdown
Contributor

@apupier I saw this is marked as draft, but CI passed, so are we ok to merge this?

@apupier
apupier marked this pull request as ready for review September 8, 2026 06:53
@apupier

apupier commented Sep 8, 2026

Copy link
Copy Markdown
Contributor Author

I saw this is marked as draft, but CI passed, so are we ok to merge this?

Yes, it is ok. I put it as draft as i didn't had time to launch thee full test suite locally, and also prefer to have a green CI before asking for a review by committers more generally

@cgivre
cgivre merged commit 0e5ea8d into apache:master Sep 8, 2026
7 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants