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 #275] Add trace message for pub and sub. #276

Merged
merged 4 commits into from
Mar 16, 2020

Conversation

ShannonDing
Copy link
Member

What is the purpose of the change

#275

feat(trace): add trace message for pub and sub.

Brief changelog

support trace send message sync
support trace consumes messages
support trace consumes orderly messages.

@ShannonDing ShannonDing added the enhancement New feature or request label Mar 16, 2020
@ShannonDing ShannonDing added this to the 2.1.0 milestone Mar 16, 2020
@ShannonDing ShannonDing linked an issue Mar 16, 2020 that may be closed by this pull request
@codecov-io
Copy link

Codecov Report

Merging #276 into master will increase coverage by 1.8%.
The diff coverage is 84.21%.

Impacted file tree graph

@@            Coverage Diff            @@
##           master     #276     +/-   ##
=========================================
+ Coverage   58.28%   60.08%   +1.8%     
=========================================
  Files         182      200     +18     
  Lines       11805    12560    +755     
=========================================
+ Hits         6880     7547    +667     
- Misses       4925     5013     +88
Impacted Files Coverage Δ
src/common/NameSpaceUtil.h 100% <ø> (ø) ⬆️
src/trace/SendMessageHook.h 50% <ø> (ø)
src/consumer/DefaultMQPushConsumerImpl.h 0% <ø> (ø) ⬆️
src/producer/DefaultMQProducer.cpp 37.61% <0%> (-1.44%) ⬇️
src/consumer/ConsumeMessageOrderlyService.cpp 0% <0%> (ø) ⬆️
src/consumer/DefaultMQPushConsumer.cpp 50.51% <0%> (-2.18%) ⬇️
src/consumer/ConsumeMessageConcurrentlyService.cpp 0% <0%> (ø) ⬆️
src/MQClientFactory.cpp 20.72% <0%> (-0.07%) ⬇️
src/trace/TraceTransferBean.h 100% <100%> (ø)
src/common/DefaultMQClient.cpp 75% <100%> (+1.27%) ⬆️
... and 43 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 753ec41...07926a1. Read the comment docs.

@ShannonDing ShannonDing changed the title feat(trace): add trace message for pub and sub. [ISSUE #275] Add trace message for pub and sub. Mar 16, 2020
@vongosling vongosling merged commit 900b937 into apache:master Mar 16, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[FEATURE]Try to support trace messages.
3 participants