Skip to content

feat: use throttled punctuator#424

Merged
kishansairam9 merged 34 commits intomainfrom
callback-registry
Oct 17, 2023
Merged

feat: use throttled punctuator#424
kishansairam9 merged 34 commits intomainfrom
callback-registry

Conversation

@kishansairam9
Copy link
Copy Markdown

No description provided.

@kishansairam9 kishansairam9 requested a review from a team as a code owner October 4, 2023 11:08
Comment thread gradle/libs.versions.toml Outdated
@kishansairam9 kishansairam9 changed the title feat: use callback registry punctuator feat: use throttled punctuator Oct 10, 2023
@github-actions

This comment has been minimized.

@codecov
Copy link
Copy Markdown

codecov Bot commented Oct 13, 2023

Codecov Report

Merging #424 (d21c403) into main (c96aaad) will increase coverage by 0.22%.
Report is 2 commits behind head on main.
The diff coverage is 82.41%.

@@             Coverage Diff              @@
##               main     #424      +/-   ##
============================================
+ Coverage     79.86%   80.08%   +0.22%     
- Complexity     1423     1424       +1     
============================================
  Files           128      128              
  Lines          5587     5565      -22     
  Branches        511      509       -2     
============================================
- Hits           4462     4457       -5     
+ Misses          891      879      -12     
+ Partials        234      229       -5     
Flag Coverage Δ
unit 80.08% <82.41%> (+0.22%) ⬆️

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

Files Coverage Δ
.../core/rawspansgrouper/RawSpanGrouperConstants.java 0.00% <ø> (ø)
...pertrace/core/rawspansgrouper/RawSpansGrouper.java 88.88% <90.00%> (ø)
...rtrace/core/rawspansgrouper/RawSpansProcessor.java 86.00% <96.55%> (+4.42%) ⬆️
...race/core/rawspansgrouper/TraceEmitPunctuator.java 81.44% <73.07%> (+6.88%) ⬆️

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

@github-actions

This comment has been minimized.

@github-actions

This comment has been minimized.

@github-actions

This comment has been minimized.

kishansairam9 added 2 commits October 13, 2023 21:07
@github-actions

This comment has been minimized.

@github-actions

This comment has been minimized.

@github-actions

This comment has been minimized.

@github-actions

This comment has been minimized.

laxmanchekka
laxmanchekka previously approved these changes Oct 17, 2023
@github-actions

This comment has been minimized.

@github-actions

This comment has been minimized.

@github-actions

This comment has been minimized.

kishansairam9 added 2 commits October 17, 2023 12:33
This reverts commit b8fd337.
@github-actions

This comment has been minimized.

@github-actions

This comment has been minimized.

@kishansairam9 kishansairam9 enabled auto-merge (squash) October 17, 2023 07:16
Copy link
Copy Markdown
Contributor

@kotharironak kotharironak left a comment

Choose a reason for hiding this comment

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

Reviewed test case issue, rest is already reviewed by Laxman.

@kishansairam9 kishansairam9 merged commit 3c038a9 into main Oct 17, 2023
@kishansairam9 kishansairam9 deleted the callback-registry branch October 17, 2023 07:16
@github-actions
Copy link
Copy Markdown

Unit Test Results

  78 files  ±0    78 suites  ±0   1m 49s ⏱️ +4s
418 tests ±0  418 ✔️ ±0  0 💤 ±0  0 ❌ ±0 

Results for commit 3c038a9. ± Comparison against base commit 7b5ec6c.

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.

3 participants