Skip to content

Fix PostCommit PortableJar Flink job - #39905

Open
Amar3tto wants to merge 1 commit into
masterfrom
portablejar-flink
Open

Fix PostCommit PortableJar Flink job#39905
Amar3tto wants to merge 1 commit into
masterfrom
portablejar-flink

Conversation

@Amar3tto

@Amar3tto Amar3tto commented Aug 27, 2026

Copy link
Copy Markdown
Collaborator

uses: ./.github/actions/setup-environment-action
with:
python-version: default
java-version: '17'

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.

I see

Exception in thread "main" java.util.ServiceConfigurationError: java.net.spi.InetAddressResolverProvider: Provider org.xbill.DNS.spi.DnsjavaInetAddressResolverProvider not found
	at java.base/java.util.ServiceLoader.fail(ServiceLoader.java:593)

due to CI bumped to Java21. This should be fixed by 'Multi-Release': 'true'. Does pin to Java17 still needed and why?

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

I tried 'Multi-Release': 'true' with Java21, the same error

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.

Ideally not pin to Java17 as we still need to fix it later on. Tested in https://github.com/apache/beam/actions/runs/33089112555

c6f125a#diff-b54459154d5fd469e3aee54c9f7fe7017242b6c289f25d7a9433e2e7a15c07ebR354

excluding 'META-INF/services/java.net.spi.InetAddressResolverProvider' from the miniCluster jar it now works

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.

Opened #39905

@Abacn Abacn mentioned this pull request Aug 28, 2026
3 tasks
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