Skip to content

chore: change log level from warn to info for stale batch processor removal#12297

Merged
LiteSun merged 1 commit into
apache:masterfrom
LiteSun:sy/update-batch-processor-manager
Jun 10, 2025
Merged

chore: change log level from warn to info for stale batch processor removal#12297
LiteSun merged 1 commit into
apache:masterfrom
LiteSun:sy/update-batch-processor-manager

Conversation

@LiteSun

@LiteSun LiteSun commented Jun 9, 2025

Copy link
Copy Markdown
Member

Description

Lower log level from warn to info when removing stale batch processors,
as this is part of normal system maintenance and involves expected cleanup of unused resources.

Which issue(s) this PR fixes:

Fixes #

Checklist

  • I have explained the need for this PR and the problem it solves
  • I have explained the changes or the new features added to this PR
  • I have added tests corresponding to this change
  • I have updated the documentation to reflect this change
  • I have verified that this change is backward compatible (If not, please discuss on the APISIX mailing list first)

@dosubot dosubot Bot added the size:XS This PR changes 0-9 lines, ignoring generated files. label Jun 9, 2025

@membphis membphis left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

it is not a bug fix.

change fix to chore

@LiteSun LiteSun changed the title fix: change log level from warn to info for stale batch processor removal chore: change log level from warn to info for stale batch processor removal Jun 10, 2025
@LiteSun
LiteSun merged commit 07de03c into apache:master Jun 10, 2025
@LiteSun
LiteSun deleted the sy/update-batch-processor-manager branch June 10, 2025 03:30
laz-xyr pushed a commit to laz-xyr/apisix that referenced this pull request Jun 19, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

size:XS This PR changes 0-9 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants