Skip to content

feat: make the assignment information become debug level#861

Merged
mattisonchao merged 2 commits intomainfrom
feat.improve.log
Jan 18, 2026
Merged

feat: make the assignment information become debug level#861
mattisonchao merged 2 commits intomainfrom
feat.improve.log

Conversation

@mattisonchao
Copy link
Copy Markdown
Member

@mattisonchao mattisonchao commented Jan 18, 2026

Motivation

Making the assignment information become a debug level to avoid tons of logs.

Modification

  • Change the log info level to debug level.

Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR changes the logging behavior for shard assignment updates to reduce log verbosity. The detailed assignment information (previous and current assignments) is now logged at Debug level only when debug logging is enabled, while a simple informational message is logged at Info level otherwise.

Changes:

  • Modified log level for shard assignment updates from Info to Debug for detailed assignment data
  • Added conditional logging that logs detailed information at Debug level when enabled, otherwise logs a simple message at Info level

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 1 out of 1 changed files in this pull request and generated no new comments.


💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@mattisonchao mattisonchao merged commit 82c5e8a into main Jan 18, 2026
13 of 14 checks passed
@mattisonchao mattisonchao deleted the feat.improve.log branch January 18, 2026 04:19
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