Skip to content
New issue

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

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

Already on GitHub? # to your account

feat(broker): introduce tracing #602

Merged
merged 1 commit into from
Nov 9, 2023
Merged

feat(broker): introduce tracing #602

merged 1 commit into from
Nov 9, 2023

Conversation

ShadowySpirits
Copy link
Contributor

No description provided.

Signed-off-by: SSpirits <admin@lv5.moe>
Copy link

codecov bot commented Nov 8, 2023

Codecov Report

Merging #602 (c52fc32) into main (635db0e) will increase coverage by 0.15%.
Report is 2 commits behind head on main.
The diff coverage is 54.65%.

Additional details and impacted files
@@             Coverage Diff              @@
##               main     #602      +/-   ##
============================================
+ Coverage     58.88%   59.03%   +0.15%     
- Complexity     1915     1936      +21     
============================================
  Files           196      199       +3     
  Lines         12837    13041     +204     
  Branches       1220     1196      -24     
============================================
+ Hits           7559     7699     +140     
- Misses         4672     4729      +57     
- Partials        606      613       +7     
Components Coverage Δ
RocketMQ Broker ∅ <ø> (∅)
RocketMQ Common 12.96% <ø> (ø)
RocketMQ Controller 50.50% <ø> (+1.30%) ⬆️
RocketMQ Proxy 43.32% <48.68%> (-1.24%) ⬇️
RocketMQ Store 78.19% <100.00%> (+0.99%) ⬆️
RocketMQ Stream 60.48% <ø> (ø)
RocketMQ Metadata 38.28% <ø> (+20.63%) ⬆️
Files Coverage Δ
.../rocketmq/proxy/service/SuspendRequestService.java 63.73% <ø> (+1.36%) ⬆️
...va/com/automq/rocketmq/store/MessageStoreImpl.java 57.33% <ø> (ø)
.../java/com/automq/rocketmq/store/S3StreamStore.java 0.00% <ø> (ø)
...rocketmq/store/queue/DefaultLogicQueueManager.java 71.42% <ø> (+0.41%) ⬆️
.../automq/rocketmq/store/queue/StreamLogicQueue.java 76.63% <ø> (+0.47%) ⬆️
...m/automq/rocketmq/store/service/ReviveService.java 85.57% <100.00%> (+7.79%) ⬆️
...xy/grpc/activity/ExtendReceiveMessageActivity.java 0.00% <0.00%> (ø)
...omq/rocketmq/proxy/service/MessageServiceImpl.java 70.88% <66.66%> (-2.30%) ⬇️
...umer/ExtendReceiveMessageResponseStreamWriter.java 0.00% <0.00%> (ø)
...va/com/automq/rocketmq/proxy/ProxyTraceAspect.java 61.11% <61.11%> (ø)

... and 61 files with indirect coverage changes

Copy link

github-actions bot commented Nov 8, 2023

Qodana for JVM

1 new problem were found

Inspection name Severity Problems
Unchecked warning 🔶 Warning 1

💡 Qodana analysis was run in the pull request mode: only the changed files were checked
☁️ View the detailed Qodana report

Contact Qodana team

Contact us at qodana-support@jetbrains.com

@ShadowySpirits ShadowySpirits merged commit 7571600 into main Nov 9, 2023
@ShadowySpirits ShadowySpirits deleted the tracing branch November 9, 2023 02:02
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant