fix: remove commented-out dead code from offset_request_handler.rs

[LakshmiSravyaVedantham]

Summary

• Remove the unused commented-out handle_get_min_offset function stub in offset_request_handler.rs
• The actual implementation is handle_get_min_offset_for_static_topic which is already in use

Closes #6332

Summary by CodeRabbit

• Chores
  • Removed unused code from the codebase.

[coderabbitai]

Walkthrough

This pull request removes a commented-out async function handle_get_min_offset from the offset request handler, eliminating unused code without introducing functional changes or affecting the active code paths.

Changes

Cohort / File(s)	Summary
Dead Code Removal rocketmq-broker/src/processor/admin_broker_processor/offset_request_handler.rs	Removed 7 lines containing a commented-out async function with no behavioral impact.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

Poem

A comment once lingered in silent repose,
Now whisked away like the wind through the rows,
The code stands cleaner, no dead weight to bear,
RocketMQ soars lighter through the digital air! 🐰✨

🚥 Pre-merge checks | ✅ 6

✅ Passed checks (6 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title check	✅ Passed	The title clearly and specifically describes the main change: removing commented-out dead code from a specific file.
Linked Issues check	✅ Passed	The PR directly addresses issue #6332 by removing the unused commented-out handle_get_min_offset function stub as requested.
Out of Scope Changes check	✅ Passed	All changes in the PR are in-scope; only the commented-out dead code function was removed with no other modifications.
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Merge Conflict Detection	✅ Passed	✅ No merge conflicts detected when merging into main

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

✨ Finishing touches

• 📝 Generate docstrings

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Post copyable unit tests in a comment

---

No actionable comments were generated in the recent review. 🎉

---

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.}

[rocketmq-rust-bot]

🔊@LakshmiSravya123 🚀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💥.

[codecov]

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 42.57%. Comparing base (61c399e) to head (7530a09).
⚠️ Report is 1 commits behind head on main.

Additional details and impacted files

@@            Coverage Diff             @@
##             main    #6333      +/-   ##
==========================================
+ Coverage   42.56%   42.57%   +0.01%     
==========================================
  Files         913      913              
  Lines      128356   128356              
==========================================
+ Hits        54633    54652      +19     
+ Misses      73723    73704      -19     

☔ 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.

[mxsm]

@LakshmiSravya123 Thanks for your contribution. LGTM