Skip to content

feat : add metrics pipeline components in hypertrace-ingester#292

Merged
kotharironak merged 2 commits intomainfrom
enable-metrics-pipeline
Dec 8, 2021
Merged

feat : add metrics pipeline components in hypertrace-ingester#292
kotharironak merged 2 commits intomainfrom
enable-metrics-pipeline

Conversation

@kotharironak
Copy link
Copy Markdown
Contributor

@kotharironak kotharironak commented Dec 7, 2021

As part of this ticket - hypertrace/hypertrace#327 - on the context of providing application metrics support, this PR,

@github-actions

This comment has been minimized.

@codecov
Copy link
Copy Markdown

codecov Bot commented Dec 7, 2021

Codecov Report

Merging #292 (040990c) into main (503b369) will decrease coverage by 0.27%.
The diff coverage is 43.58%.

Impacted file tree graph

@@             Coverage Diff              @@
##               main     #292      +/-   ##
============================================
- Coverage     79.87%   79.60%   -0.28%     
- Complexity     1285     1287       +2     
============================================
  Files           116      116              
  Lines          5108     5128      +20     
  Branches        458      465       +7     
============================================
+ Hits           4080     4082       +2     
- Misses          820      836      +16     
- Partials        208      210       +2     
Flag Coverage Δ
unit 79.60% <43.58%> (-0.28%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files Coverage Δ
...va/org/hypertrace/ingester/HypertraceIngester.java 50.00% <40.54%> (-13.24%) ⬇️
...hypertrace/metrics/generator/MetricsGenerator.java 74.35% <100.00%> (ø)
...hypertrace/metrics/generator/MetricsProcessor.java 91.30% <100.00%> (ø)
...race/core/rawspansgrouper/TraceEmitPunctuator.java 75.43% <0.00%> (+0.87%) ⬆️

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 503b369...040990c. Read the comment docs.

@kotharironak kotharironak merged commit 181e8b3 into main Dec 8, 2021
@kotharironak kotharironak deleted the enable-metrics-pipeline branch December 8, 2021 12:32
@github-actions
Copy link
Copy Markdown

github-actions Bot commented Dec 8, 2021

Unit Test Results

  75 files  ±0    75 suites  ±0   1m 14s ⏱️ +5s
393 tests ±0  393 ✔️ ±0  0 💤 ±0  0 ❌ ±0 

Results for commit 181e8b3. ± Comparison against base commit 503b369.

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.

2 participants