Skip to content

Conversation

Zylphrex
Copy link
Member

@Zylphrex Zylphrex commented Oct 3, 2025

This allows count_unique to take numeric attributes as an argument.

Backend for LOGS-398

This allows count_unique to take numeric attributes as an argument.
@Zylphrex Zylphrex requested a review from a team as a code owner October 3, 2025 16:35
@github-actions github-actions bot added the Scope: Backend Automatically applied to PRs that change backend components label Oct 3, 2025
Copy link

codecov bot commented Oct 3, 2025

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   #100889      +/-   ##
===========================================
- Coverage   81.15%    81.14%   -0.01%     
===========================================
  Files        8615      8615              
  Lines      382250    382256       +6     
  Branches    24033     24033              
===========================================
- Hits       310199    310198       -1     
- Misses      71723     71730       +7     
  Partials      328       328              

Zylphrex added a commit that referenced this pull request Oct 3, 2025
Numeric attributes should be permitted as an argument to count_unique.

Requires #100889

Frontend for LOGS-398
@Zylphrex Zylphrex merged commit c86e584 into master Oct 3, 2025
67 checks passed
@Zylphrex Zylphrex deleted the txiao/feat/support-numeric-values-in-eap-count-unique branch October 3, 2025 17:31
jasonyuezhang pushed a commit to jasonyuezhang/sentry that referenced this pull request Oct 3, 2025
Numeric attributes should be permitted as an argument to count_unique.

Requires getsentry#100889

Frontend for LOGS-398
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Scope: Backend Automatically applied to PRs that change backend components
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants