[ISSUE #6499]✨Remove useless code from trigger.rs

[MasterOogway1466]

Which Issue(s) This PR Fixes(Closes)

• Fixes [Enhancement✨] Remove useless code from trigger.rs #6499

Brief Description

How Did You Test This Change?

Summary by CodeRabbit

• Refactor
  • Improved internal time conversion handling for better code maintainability. No user-facing changes.

[rocketmq-rust-bot]

🔊@MasterOogway1466 🚀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

Refactors the system_time_to_datetime function in the schedule module by replacing manual SystemTime to DateTime<Utc> conversion logic with a direct DateTime::from() call, eliminating 3 lines of redundant code while maintaining equivalent behavior.

Changes

Cohort / File(s)	Summary
Code Simplification rocketmq/src/schedule/trigger.rs	Replaced manual SystemTime-to-DateTime conversion with idiomatic DateTime::from(system_time) call, removing 3 lines of code.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Poem

🐰 A hop through the code, what a delight,
Cleaning up conversions, making it right,
From trait so elegant, short and so neat,
Three fewer lines make the refactor complete! ✨

🚥 Pre-merge checks | ✅ 5

✅ Passed checks (5 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Linked Issues check	✅ Passed	The PR successfully addresses issue #6499 by removing unnecessary code from trigger.rs, simplifying the conversion logic as intended.
Out of Scope Changes check	✅ Passed	All changes are focused on the stated objective of removing useless code from trigger.rs; no out-of-scope modifications are present.
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Title check	✅ Passed	The title accurately describes the main change: removing useless code (specifically simplifying a conversion call) from trigger.rs by delegating to a From implementation.

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

✨ Finishing Touches

• 📝 Generate docstrings (stacked PR)
• 📝 Generate docstrings (commit on current branch)

🧪 Generate unit tests (beta)

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

---

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

[codecov]

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 42.18%. Comparing base (51c7663) to head (81263ef).
⚠️ Report is 1 commits behind head on main.

Additional details and impacted files

@@           Coverage Diff           @@
##             main    #6500   +/-   ##
=======================================
  Coverage   42.18%   42.18%           
=======================================
  Files         942      942           
  Lines      131785   131785           
=======================================
+ Hits        55592    55593    +1     
+ Misses      76193    76192    -1     

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

[rocketmq-rust-bot]

LGTM - All CI checks passed ✅

[mxsm]

LGTM