Skip to content

Conversation

@kotharironak
Copy link
Contributor

This PR,

  • adds support for span drop filter criteria for operation ame
  • adds exits/not_exits operator

skjindal93
skjindal93 previously approved these changes Dec 27, 2022
@github-actions

This comment has been minimized.

@codecov
Copy link

codecov bot commented Dec 27, 2022

Codecov Report

Merging #364 (7805d53) into main (ed7b5b9) will decrease coverage by 0.10%.
The diff coverage is 56.00%.

@@             Coverage Diff              @@
##               main     #364      +/-   ##
============================================
- Coverage     79.68%   79.58%   -0.11%     
- Complexity     1389     1409      +20     
============================================
  Files           126      126              
  Lines          5509     5531      +22     
  Branches        504      520      +16     
============================================
+ Hits           4390     4402      +12     
- Misses          889      893       +4     
- Partials        230      236       +6     
Flag Coverage Δ
unit 79.58% <56.00%> (-0.11%) ⬇️

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

Impacted Files Coverage Δ
...ertrace/core/spannormalizer/jaeger/SpanFilter.java 81.19% <54.16%> (-7.35%) ⬇️
...ace/core/spannormalizer/jaeger/SpanDropFilter.java 89.47% <100.00%> (+0.58%) ⬆️
...race/core/rawspansgrouper/TraceEmitPunctuator.java 75.43% <0.00%> (+0.87%) ⬆️

📣 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.

@skjindal93 skjindal93 merged commit b774e40 into main Dec 27, 2022
@skjindal93 skjindal93 deleted the enhance-drop-filter branch December 27, 2022 18:24
@github-actions
Copy link

Unit Test Results

  76 files  ±0    76 suites  ±0   1m 9s ⏱️ -29s
413 tests ±0  413 ✔️ ±0  0 💤 ±0  0 ❌ ±0 

Results for commit b774e40. ± Comparison against base commit ed7b5b9.

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