Skip to content

[improve][doc] Add breaking change note for JUL-to-Log4j2 bridge in 5.0.0-M2 - #1229

Merged
lhotari merged 2 commits into
apache:mainfrom
geniusjoe:doc/jul-bridge-migration-guide
Sep 17, 2026
Merged

lhotari merged 2 commits into
apache:mainfrom
geniusjoe:doc/jul-bridge-migration-guide

Conversation

@geniusjoe

@geniusjoe geniusjoe commented Sep 17, 2026 •

Copy link
Copy Markdown
Contributor

Motivation

PR #26330 bridges JUL (java.util.logging) to Log4j2 in Pulsar 5.0.0-M2. This is a breaking change because existing JUL configurations and APIs (e.g., -Djava.util.logging.config.file, Logger.setLevel(), Logger.addHandler()) will no longer take effect after the upgrade.

The current 5.0.0-M2 release notes only list this PR under the ### Broker section without highlighting the breaking nature of the change. Users upgrading to 5.0.0-M2 need a clear heads-up with migration guidance.

Modifications

Added a ### Breaking changes section to release-notes/versioned/pulsar-5.0.0-M2.md (between Approved PIPs and Library updates), including:

  • A summary of why JUL configuration is no longer effective
  • A list of affected JUL APIs and configurations
  • Migration instructions (move JUL config to conf/log4j2.yaml)
  • Revert instructions (PULSAR_EXTRA_OPTS to restore stock JUL behavior)

Preview

image WXWorkCapture_17896216166092

✅ Contribution Checklist

@geniusjoe
geniusjoe requested a review from lhotari September 17, 2026 07:03
@geniusjoe

Copy link
Copy Markdown
Contributor Author

Hi @lhotari , I've drafted a breaking change note for apache/pulsar#26330 in the 5.0.0-M2 release notes, covering the affected JUL APIs, migration guide, and revert instructions. Could you take a look and let me know if anything needs to be adjusted? Also, if it looks good, should we add the same note to the GitHub release page at https://github.com/apache/pulsar/releases/tag/v5.0.0-M2 as well?

Comment thread release-notes/versioned/pulsar-5.0.0-M2.md Outdated
@lhotari
lhotari merged commit 1dbff6b into apache:main Sep 17, 2026
2 checks passed
@geniusjoe

Copy link
Copy Markdown
Contributor Author

I've also added the same breaking change note to the GitHub release page. https://github.com/apache/pulsar/releases/tag/v5.0.0-M2

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants