Skip to content

[ISSUE #9249] When delivery fails, there is an incorrect start offset in the delivery settings#9252

Merged
lizhimins merged 1 commit intoapache:developfrom
coolmoon101:fix-bug-9249
Mar 17, 2025
Merged

[ISSUE #9249] When delivery fails, there is an incorrect start offset in the delivery settings#9252
lizhimins merged 1 commit intoapache:developfrom
coolmoon101:fix-bug-9249

Conversation

@coolmoon101
Copy link
Copy Markdown
Contributor

5.3.2版本,延迟消息,投递失败,投递设置的开始offset有误。
具体issues地址:#9249

…Offset to currOffset when delivery fails, otherwise the currOffset message is lost.
@codecov-commenter
Copy link
Copy Markdown

codecov-commenter commented Mar 17, 2025

Codecov Report

❌ Patch coverage is 0% with 1 line in your changes missing coverage. Please review.
✅ Project coverage is 48.04%. Comparing base (054f910) to head (441cdfd).
⚠️ Report is 168 commits behind head on develop.

Files with missing lines Patch % Lines
...cketmq/broker/schedule/ScheduleMessageService.java 0.00% 1 Missing ⚠️
Additional details and impacted files
@@              Coverage Diff              @@
##             develop    #9252      +/-   ##
=============================================
- Coverage      48.08%   48.04%   -0.05%     
+ Complexity     12110    12094      -16     
=============================================
  Files           1322     1322              
  Lines          93115    93115              
  Branches       11938    11938              
=============================================
- Hits           44777    44739      -38     
- Misses         42800    42834      +34     
- Partials        5538     5542       +4     

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

@RongtongJin RongtongJin changed the title [ISSUE apache#9249]延迟消息,投递失败,投递设置的开始offset有误修改 [ISSUE #9249] When delivery fails, there is an incorrect start offset in the delivery settings Mar 17, 2025
@lizhimins lizhimins merged commit b695652 into apache:develop Mar 17, 2025
10 of 11 checks passed
@coolmoon101
Copy link
Copy Markdown
Contributor Author

代码已经合了,原来的issue#9249,我需要去关闭吗 @yuz10

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.

5 participants