[ISSUE #5533]✨Enhance logging in ClientRequestProcessor by adding debug level for request code

[WaterWhisperer]

Which Issue(s) This PR Fixes(Closes)

Fixes #5533

Brief Description

How Did You Test This Change?

Summary by CodeRabbit

• Chores
  • Adjusted internal logging levels for improved operational efficiency.

_{✏️ Tip: You can customize this high-level summary in your review settings.}

[rocketmq-rust-bot]

🔊@WaterWhisperer 🚀Thanks for your contribution🎉！

💡CodeRabbit(AI) will review your code first🔥！

Note

🚨The code review suggestions from CodeRabbit are to be used as a reference only, and the PR submitter can decide whether to make changes based on their own judgment. Ultimately, the project management personnel will conduct the final code review💥.

[coderabbitai]

Walkthrough

The change modifies logging verbosity in the ClientRequestProcessor by downgrading the request code logging statement from info to debug level, reducing noise in standard logs while retaining detailed information for debug-level output.

Changes

Cohort / File(s)	Summary
Logging Level Adjustment rocketmq-namesrv/src/processor/client_request_processor.rs	Import statement changed from use tracing::info; to use tracing::debug;; corresponding log macro invocation updated from info!(...) to debug!(...) for request code visibility

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

Poem

🐰 A whisper quieter than before,
Debug logs now guard the door,
Info flurry turned to peace,
Request codes seek their release,
Cleaner logs, oh what a feat! 📝

🚥 Pre-merge checks | ✅ 5

✅ Passed checks (5 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title check	✅ Passed	The title accurately reflects the main change: replacing info-level logging with debug-level logging for request code in ClientRequestProcessor, directly addressing the linked issue.
Linked Issues check	✅ Passed	The code changes successfully implement the objective from issue #5533: adding debug-level logging for request codes in ClientRequestProcessor by replacing info with debug.
Out of Scope Changes check	✅ Passed	All changes are focused on the logging level modification in ClientRequestProcessor; no unrelated or out-of-scope code changes are present.
Docstring Coverage	✅ Passed	Docstring coverage is 100.00% which is sufficient. The required threshold is 80.00%.

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

✨ Finishing touches

• 📝 Generate docstrings

---

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

• X
• Mastodon
• Reddit
• LinkedIn

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

[coderabbitai]

Actionable comments posted: 1

🤖 Fix all issues with AI agents

In @rocketmq-namesrv/src/processor/client_request_processor.rs:
- Around line 57-60: The per-request log in ClientRequestProcessor was lowered
from info! to debug!, but DefaultRequestProcessor still logs at info!, so update
the equivalent log call in DefaultRequestProcessor to use debug! as well (locate
the Info log in the DefaultRequestProcessor implementation—e.g., inside the
request handling method where it prints "Name server DefaultRequestProcessor
Received request code: {:?}") and change info! to debug! so both processors use
the same logging level and message pattern for per-request events.

📜 Review details

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between a6f6cb2 and 0f1cc9b.

📒 Files selected for processing (1)

• rocketmq-namesrv/src/processor/client_request_processor.rs

⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (5)

• GitHub Check: Code Coverage
• GitHub Check: Build & Test (windows-latest)
• GitHub Check: Build & Test (ubuntu-latest)
• GitHub Check: Build & Test (macos-latest)
• GitHub Check: auto-approve

🔇 Additional comments (1)

rocketmq-namesrv/src/processor/client_request_processor.rs (1)

31-31: LGTM: Import correctly updated to support debug logging.

The import change from tracing::info to tracing::debug is correct and aligns with the updated log statement at line 57.

[mxsm]

@WaterWhisperer I think you could take on some more challenging issues, with difficulty levels ranging from medium to hard, or share some of your own ideas and optimizations for the project.Improving one's own abilities in this way would be better

[mxsm]

LGTM

[codecov]

Codecov Report

❌ Patch coverage is 0% with 1 line in your changes missing coverage. Please review.
✅ Project coverage is 38.43%. Comparing base (a6f6cb2) to head (0f1cc9b).
⚠️ Report is 1 commits behind head on main.

Files with missing lines	Patch %	Lines
...-namesrv/src/processor/client_request_processor.rs	0.00%	1 Missing ⚠️

Additional details and impacted files

@@           Coverage Diff           @@
##             main    #5546   +/-   ##
=======================================
  Coverage   38.43%   38.43%           
=======================================
  Files         815      815           
  Lines      110551   110551           
=======================================
  Hits        42492    42492           
  Misses      68059    68059           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
• 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

[WaterWhisperer]

@WaterWhisperer I think you could take on some more challenging issues, with difficulty levels ranging from medium to hard, or share some of your own ideas and optimizations for the project.Improving one's own abilities in this way would be better

@mxsm Thanks, I always wanted to try harder issues, and they were claimed when I saw them before