Skip to content

Add class introduced in 5.14.1/6.0.1#794

Merged
jormundur00 merged 1 commit intograalvm:masterfrom
marcphilipp:patch-3
Mar 30, 2026
Merged

Add class introduced in 5.14.1/6.0.1#794
jormundur00 merged 1 commit intograalvm:masterfrom
marcphilipp:patch-3

Conversation

@marcphilipp
Copy link
Copy Markdown
Contributor

MethodSegmentResolver is used during discovery and therefore also needs to be initialized at build time on JDK <= 21.

`MethodSegmentResolver` is used during discovery and therefore also needs to be initialized at build time on JDK <= 21.
@marcphilipp
Copy link
Copy Markdown
Contributor Author

@dnestoro Could you please take a look?

@phipag
Copy link
Copy Markdown

phipag commented Jan 19, 2026

Hi all, I would love a review here since it affects my native tests. See failing CI here: aws-powertools/powertools-lambda-java#2267

@marcphilipp
Copy link
Copy Markdown
Contributor Author

@melix Could you please take a look?

@marcphilipp
Copy link
Copy Markdown
Contributor Author

@melix Could you please merge this?

@melix
Copy link
Copy Markdown
Collaborator

melix commented Jan 26, 2026

@vjovanov please confirm and merge if you agree :)

@phipag
Copy link
Copy Markdown

phipag commented Mar 17, 2026

Hey @melix @vjovanov @fniephaus. Can you please have a look here and merge this PR? This is currently failing the native unit tests in our CI.

Example: https://github.com/aws-powertools/powertools-lambda-java/actions/runs/22666981417/job/65701100409?pr=2404

Copy link
Copy Markdown
Member

@jormundur00 jormundur00 left a comment

Choose a reason for hiding this comment

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

Took a look at org.junit.jupiter.engine.discovery.MethodSegmentResolver and it looks safe to add to the initialize-at-buildtime list. LGTM.

@jormundur00 jormundur00 merged commit c6f3674 into graalvm:master Mar 30, 2026
190 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

OCA Verified All contributors have signed the Oracle Contributor Agreement.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants