You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@rocketmq.apache.org by "lizhimins (via GitHub)" <gi...@apache.org> on 2025/02/27 11:23:00 UTC
[PR] Revert "[ISSUE #9176] Fix pop message header missing fields when enable ACL 2.0" [rocketmq]
lizhimins opened a new pull request, #9211:
URL: https://github.com/apache/rocketmq/pull/9211
When there is a time difference between the client and the server, it can cause issues with suspending long-polling POP requests.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: commits-unsubscribe@rocketmq.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] Revert "[ISSUE #9176] Fix pop message header missing fields when enable ACL 2.0" [rocketmq]
Posted by "codecov-commenter (via GitHub)" <gi...@apache.org>.
codecov-commenter commented on PR #9211:
URL: https://github.com/apache/rocketmq/pull/9211#issuecomment-2687729014
## [Codecov](https://app.codecov.io/gh/apache/rocketmq/pull/9211?dropdown=coverage&src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) Report
All modified and coverable lines are covered by tests :white_check_mark:
> Project coverage is 48.08%. Comparing base [(`19f799d`)](https://app.codecov.io/gh/apache/rocketmq/commit/19f799d93617c17b093c9e372c5141f3ce32b292?dropdown=coverage&el=desc&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) to head [(`c909084`)](https://app.codecov.io/gh/apache/rocketmq/commit/c909084094ddf623c474cf0a22a5770997451463?dropdown=coverage&el=desc&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache).
<details><summary>Additional details and impacted files</summary>
```diff
@@ Coverage Diff @@
## develop #9211 +/- ##
=============================================
- Coverage 48.10% 48.08% -0.03%
+ Complexity 12106 12095 -11
=============================================
Files 1321 1321
Lines 93081 93080 -1
Branches 11933 11933
=============================================
- Hits 44779 44754 -25
- Misses 42773 42788 +15
- Partials 5529 5538 +9
```
</details>
[:umbrella: View full report in Codecov by Sentry](https://app.codecov.io/gh/apache/rocketmq/pull/9211?dropdown=coverage&src=pr&el=continue&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache).
:loudspeaker: Have feedback on the report? [Share it here](https://about.codecov.io/codecov-pr-comment-feedback/?utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache).
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: commits-unsubscribe@rocketmq.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] Revert "[ISSUE #9176] Fix pop message header missing fields when enable ACL 2.0" [rocketmq]
Posted by "lollipopjin (via GitHub)" <gi...@apache.org>.
lollipopjin merged PR #9211:
URL: https://github.com/apache/rocketmq/pull/9211
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: commits-unsubscribe@rocketmq.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org