Skip to content

[ISSUE #9860] Honor enablePopLog for routine revive logs - #11236

Open
yym108 wants to merge 1 commit into
apache:developfrom
yym108:fix/issue-9860-pop-revive-logs
Open

yym108 wants to merge 1 commit into
apache:developfrom
yym108:fix/issue-9860-pop-revive-logs

Conversation

@yym108

@yym108 yym108 commented Sep 26, 2026

Copy link
Copy Markdown

Which Issue(s) This PR Fixes

Brief Description

Routine POP revive progress messages are written at INFO on every cycle even when enablePopLog is false. Gate seven routine log sites in PopReviveService with the existing setting, covering start/skip, offsets, checkpoint summaries, completion and idle cycles. This lets the default configuration suppress repetitive progress logging while retaining the diagnostic messages when enabled.

How Did You Test This Change?

  • PopReviveServiceTest: all 12 existing tests passed on macOS ARM64 with Zulu JDK 21.
  • Reviewed the seven logging guards; the existing tests exercise revive behavior and do not directly assert captured log output.
  • Maven's configured Checkstyle/SpotBugs checks and git diff --check passed.
  • Local JDK 21 execution used JaCoCo 0.8.12, Javassist 3.30.2-GA, Byte Buddy experimental support and JDK module opens. No build configuration changes are included.

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.

[Feature] 移除不必要的info日志打印

1 participant