Skip to content

Conversation

JayFrank
Copy link
Contributor

@JayFrank JayFrank commented Dec 2, 2021

完善BrokerServer部分描述细节

@coveralls
Copy link

Coverage Status

Coverage increased (+0.006%) to 54.387% when pulling f1ee023 on JayFrank:patch-2 into a75c742 on apache:master.

@codecov-commenter
Copy link

Codecov Report

Merging #3573 (f1ee023) into master (a75c742) will decrease coverage by 0.12%.
The diff coverage is n/a.

Impacted file tree graph

@@             Coverage Diff              @@
##             master    #3573      +/-   ##
============================================
- Coverage     48.31%   48.18%   -0.13%     
+ Complexity     4611     4600      -11     
============================================
  Files           555      555              
  Lines         36756    36756              
  Branches       4841     4841              
============================================
- Hits          17759    17712      -47     
- Misses        16772    16813      +41     
- Partials       2225     2231       +6     
Impacted Files Coverage Δ
...rg/apache/rocketmq/common/stats/StatsSnapshot.java 84.61% <0.00%> (-15.39%) ⬇️
...in/java/org/apache/rocketmq/test/util/MQAdmin.java 38.88% <0.00%> (-5.56%) ⬇️
...org/apache/rocketmq/store/ha/WaitNotifyObject.java 66.07% <0.00%> (-5.36%) ⬇️
...va/org/apache/rocketmq/common/stats/StatsItem.java 50.00% <0.00%> (-5.00%) ⬇️
...nt/impl/consumer/ConsumeMessageOrderlyService.java 38.98% <0.00%> (-3.62%) ⬇️
...he/rocketmq/client/impl/consumer/ProcessQueue.java 57.67% <0.00%> (-3.26%) ⬇️
...org/apache/rocketmq/common/stats/StatsItemSet.java 41.79% <0.00%> (-2.99%) ⬇️
...a/org/apache/rocketmq/filter/util/BloomFilter.java 59.13% <0.00%> (-2.16%) ⬇️
...a/org/apache/rocketmq/store/StoreStatsService.java 29.50% <0.00%> (-1.32%) ⬇️
...pl/consumer/ConsumeMessageConcurrentlyService.java 47.57% <0.00%> (-0.98%) ⬇️
... and 6 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 a75c742...f1ee023. Read the comment docs.

@vongosling vongosling changed the title 完善BrokerServer部分描述细节 Polish the doc for tracing parts. Dec 7, 2021
@RongtongJin RongtongJin changed the base branch from master to develop December 7, 2021 03:19

4. DefaultMQProducer

`DefaultMQProducer(final String producerGroup, boolean enableMsgTrace, final String customizedTraceTopic)`

使用指定的分组名创建一个生产者,并设置是否开启消息追踪及追踪topic的名称
使用指定的分组名创建一个生产者,并设置是否开启消息轨迹及追踪topic的名称
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

“追踪topic”与下文 “轨迹topic”不一致

@yuz10 yuz10 added this to the 4.9.3 milestone Dec 10, 2021
@yuz10 yuz10 merged commit 8b58328 into apache:develop Dec 10, 2021
@francisoliverlee francisoliverlee changed the title Polish the doc for tracing parts. [ISSUE #3424] Polish the doc for tracing parts. Feb 21, 2022
devfat pushed a commit to devfat/rocketmq that referenced this pull request Mar 3, 2022
* fix(broker): can not clear topic unitSubFlag (apache#3429)

Co-authored-by: lushilin <[email protected]>

* doc-optimization (apache#3523)

* 完善BrokerServer部分描述细节

完善BrokerServer部分描述细节

Co-authored-by: lushilin <[email protected]>
Co-authored-by: lushilin <[email protected]>
Co-authored-by: Smile <[email protected]>
GenerousMan pushed a commit to GenerousMan/rocketmq that referenced this pull request Aug 12, 2022
* fix(broker): can not clear topic unitSubFlag (apache#3429)

Co-authored-by: lushilin <[email protected]>

* doc-optimization (apache#3523)

* 完善BrokerServer部分描述细节

完善BrokerServer部分描述细节

Co-authored-by: lushilin <[email protected]>
Co-authored-by: lushilin <[email protected]>
Co-authored-by: Smile <[email protected]>
carlvine500 pushed a commit to carlvine500/rocketmq-apache that referenced this pull request Sep 10, 2024
* fix(broker): can not clear topic unitSubFlag (apache#3429)

Co-authored-by: lushilin <[email protected]>

* doc-optimization (apache#3523)

* 完善BrokerServer部分描述细节

完善BrokerServer部分描述细节

Co-authored-by: lushilin <[email protected]>
Co-authored-by: lushilin <[email protected]>
Co-authored-by: Smile <[email protected]>
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.

6 participants