Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[improve] Support overriding java.net.preferIPv4Stack with OPTS #23846

Merged

Conversation

lhotari
Copy link
Member

@lhotari lhotari commented Jan 14, 2025

Motivation

When using Pulsar with IPv6, it's necessary to set -Djava.net.preferIPv4Stack=false since Pulsar startup scripts set -Djava.net.preferIPv4Stack=true. Currently it's not possible to override the Pulsar default by passing -Djava.net.preferIPv4Stack=false in OPTS environment variable. This PR intends to make it possible.
This PR is related to the issue #23843.

Modifications

Prepend the default value -Djava.net.preferIPv4Stack=true to OPTS instead of appending to it so that the setting in OPTS takes preference. The last setting in the argument line wins when setting system properties for Java processes.

Documentation

  • doc
  • doc-required
  • doc-not-needed
  • doc-complete

@lhotari lhotari added this to the 4.1.0 milestone Jan 14, 2025
@lhotari lhotari self-assigned this Jan 14, 2025
@github-actions github-actions bot added the doc-not-needed Your PR changes do not impact docs label Jan 14, 2025
Copy link
Member

@nodece nodece left a comment

Choose a reason for hiding this comment

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

LGTM

Copy link
Contributor

@nicoloboschi nicoloboschi left a comment

Choose a reason for hiding this comment

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

+1

@nodece
Copy link
Member

nodece commented Jan 14, 2025

/pulsarbot rerun-failure-checks

@lhotari lhotari merged commit aff9ec8 into apache:master Jan 14, 2025
59 of 61 checks passed
lhotari added a commit that referenced this pull request Jan 16, 2025
lhotari added a commit that referenced this pull request Jan 17, 2025
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.

3 participants