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

fix(eap): Count takes arg #79066

Merged
merged 2 commits into from
Oct 15, 2024
Merged

fix(eap): Count takes arg #79066

merged 2 commits into from
Oct 15, 2024

Conversation

Zylphrex
Copy link
Member

Unlike before, count() will now take an argument to count the numeric attribute with a default to count span.duration.

Unlike before, count() will now take an argument to count the numeric attribute
with a default to count span.duration.
@github-actions github-actions bot added the Scope: Backend Automatically applied to PRs that change backend components label Oct 11, 2024
Copy link

codecov bot commented Oct 11, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

✅ All tests successful. No failed tests found.

Additional details and impacted files
@@           Coverage Diff           @@
##           master   #79066   +/-   ##
=======================================
  Coverage   78.28%   78.29%           
=======================================
  Files        7129     7132    +3     
  Lines      313857   313925   +68     
  Branches    51234    51253   +19     
=======================================
+ Hits       245705   245774   +69     
+ Misses      61724    61721    -3     
- Partials     6428     6430    +2     

@work-cpu
Copy link

/assign

@Zylphrex Zylphrex merged commit 5763714 into master Oct 15, 2024
50 checks passed
@Zylphrex Zylphrex deleted the txiao/fix/eap-count-takes-arg branch October 15, 2024 14:46
cmanallen pushed a commit that referenced this pull request Oct 23, 2024
Unlike before, count() will now take an argument to count the numeric
attribute with a default to count `span.duration`.
@github-actions github-actions bot locked and limited conversation to collaborators Oct 31, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Scope: Backend Automatically applied to PRs that change backend components
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants