Skip to content

OF-3304: Demote and improve MUC history reload limit log message#3383

Open
akrherz wants to merge 1 commit into
igniterealtime:mainfrom
akrherz:of3304_improve_log_msg
Open

OF-3304: Demote and improve MUC history reload limit log message#3383
akrherz wants to merge 1 commit into
igniterealtime:mainfrom
akrherz:of3304_improve_log_msg

Conversation

@akrherz

@akrherz akrherz commented Jun 5, 2026

Copy link
Copy Markdown
Member

No description provided.

@coderabbitai

coderabbitai Bot commented Jun 5, 2026

Copy link
Copy Markdown

Ready to act? Review this PR in Change Stack to turn feedback into patch suggestions you can inspect and refine.

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Repository UI

Review profile: CHILL

Plan: Pro Plus

Run ID: f42bc02e-e93d-4d82-94f2-6e79c778e593

📥 Commits

Reviewing files that changed from the base of the PR and between 75c25b0 and c908062.

📒 Files selected for processing (1)
  • xmppserver/src/main/java/org/jivesoftware/openfire/muc/spi/MUCPersistenceManager.java

📝 Walkthrough

Walkthrough

This pull request updates the MUCPersistenceManager source file with two minimal changes. The copyright year in the file header is incremented from 2025 to 2026. In the room history reload functionality, the log output for the reload limit is changed from a warning-level message to an info-level message, with the same computed reload-limit-days parameter value.

🚥 Pre-merge checks | ✅ 2 | ❌ 1

❌ Failed checks (1 inconclusive)

Check name Status Explanation Resolution
Description check ❓ Inconclusive No pull request description was provided by the author, but the changeset is clearly related to the PR objectives which indicate improved log messaging for MUC history reload. Add a brief description explaining the changes, such as 'Demote MUC history reload limit log message from warning to info level and improve formatting.'
✅ Passed checks (2 passed)
Check name Status Explanation
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.


Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

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.

1 participant