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

[INLONG-2176] Add histogram metric and client-side metric output #2177

Merged
merged 3 commits into from
Jan 18, 2022
Merged

[INLONG-2176] Add histogram metric and client-side metric output #2177

merged 3 commits into from
Jan 18, 2022

Conversation

gosonzhang
Copy link
Contributor

Fixes #2176

@codecov-commenter
Copy link

Codecov Report

Merging #2177 (ebf0ab4) into master (c2439b5) will increase coverage by 0.01%.
The diff coverage is 30.94%.

Impacted file tree graph

@@             Coverage Diff              @@
##             master    #2177      +/-   ##
============================================
+ Coverage     12.57%   12.58%   +0.01%     
- Complexity     1194     1208      +14     
============================================
  Files           414      417       +3     
  Lines         35282    35735     +453     
  Branches       5546     5590      +44     
============================================
+ Hits           4436     4499      +63     
- Misses        30049    30435     +386     
- Partials        797      801       +4     
Impacted Files Coverage Δ
.../inlong/tubemq/client/common/TClientConstants.java 66.66% <ø> (ø)
...q/client/consumer/SimpleClientBalanceConsumer.java 0.00% <0.00%> (ø)
...emq/client/consumer/SimplePullMessageConsumer.java 2.88% <0.00%> (-0.06%) ⬇️
...emq/client/consumer/SimplePushMessageConsumer.java 8.39% <0.00%> (-0.14%) ⬇️
.../tubemq/client/producer/SimpleMessageProducer.java 12.50% <0.00%> (-0.57%) ⬇️
...e/inlong/tubemq/corebase/metric/AbsMetricItem.java 75.00% <0.00%> (ø)
...ng/tubemq/corebase/metric/GaugeNormMetricItem.java 100.00% <ø> (ø)
...ng/tubemq/client/consumer/BaseMessageConsumer.java 6.20% <3.70%> (-0.05%) ⬇️
...inlong/tubemq/client/producer/ProducerManager.java 14.21% <9.09%> (+0.03%) ⬆️
.../inlong/tubemq/client/config/TubeClientConfig.java 12.80% <13.04%> (+0.02%) ⬆️
... and 14 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 c2439b5...ebf0ab4. Read the comment docs.

@dockerzhang dockerzhang merged commit e1235ad into apache:master Jan 18, 2022
@gosonzhang gosonzhang deleted the INLONG-2176 branch January 18, 2022 09:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Add histogram metric and client-side metric output
4 participants