Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[ISSUE #4612] Fix trace not complete #6941

Merged
merged 1 commit into from
Jun 25, 2023
Merged

[ISSUE #4612] Fix trace not complete #6941

merged 1 commit into from
Jun 25, 2023

Conversation

panzhi33
Copy link
Contributor

@panzhi33 panzhi33 commented Jun 25, 2023

Which Issue(s) This PR Fixes

Fixes #4612

Brief Description

How Did You Test This Change?

@codecov-commenter
Copy link

Codecov Report

Merging #6941 (55028f2) into develop (e369d7d) will increase coverage by 0.00%.
The diff coverage is 75.00%.

@@            Coverage Diff             @@
##             develop    #6941   +/-   ##
==========================================
  Coverage      42.66%   42.66%           
- Complexity      9207     9217   +10     
==========================================
  Files           1131     1131           
  Lines          80421    80434   +13     
  Branches       10511    10512    +1     
==========================================
+ Hits           34310    34316    +6     
- Misses         41822    41834   +12     
+ Partials        4289     4284    -5     
Impacted Files Coverage Δ
...nt/impl/consumer/ConsumeMessageOrderlyService.java 49.46% <0.00%> (-0.18%) ⬇️
...consumer/ConsumeMessagePopConcurrentlyService.java 5.67% <0.00%> (-0.03%) ⬇️
...lient/impl/consumer/DefaultMQPullConsumerImpl.java 29.32% <0.00%> (-0.09%) ⬇️
...apache/rocketmq/client/trace/TraceDataEncoder.java 69.28% <75.00%> (-0.46%) ⬇️
...he/rocketmq/client/hook/ConsumeMessageContext.java 89.28% <100.00%> (+1.28%) ⬆️
...pl/consumer/ConsumeMessageConcurrentlyService.java 48.85% <100.00%> (+0.23%) ⬆️
...ent/impl/consumer/DefaultLitePullConsumerImpl.java 68.61% <100.00%> (-0.42%) ⬇️
...org/apache/rocketmq/client/trace/TraceContext.java 75.00% <100.00%> (+1.53%) ⬆️
...client/trace/hook/ConsumeMessageTraceHookImpl.java 78.57% <100.00%> (+0.38%) ⬆️

... and 15 files with indirect coverage changes

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

@RongtongJin RongtongJin changed the title [ISSUE #4612] fix trace not complete [ISSUE #4612] Fix trace not complete Jun 25, 2023
@panzhi33 panzhi33 closed this Jun 25, 2023
@RongtongJin RongtongJin merged commit 16ef575 into apache:develop Jun 25, 2023
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.

fix trace not complete
3 participants