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

feat(rpc): Support RPC aggregate filters #83590

Merged
merged 3 commits into from
Jan 17, 2025

Conversation

Zylphrex
Copy link
Member

This adds support for aggregate filters for the table RPC.

Requires getsentry/snuba#6776.

This adds support for aggregate filters for the table RPC. Requires getsentry/snuba#6776.
@Zylphrex Zylphrex requested a review from a team as a code owner January 16, 2025 17:02
@github-actions github-actions bot added the Scope: Backend Automatically applied to PRs that change backend components label Jan 16, 2025
Copy link
Member

@wmak wmak left a comment

Choose a reason for hiding this comment

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

Can we add a few SearchResolver unit tests too? otherwise lgtm

Copy link

codecov bot commented Jan 16, 2025

Codecov Report

Attention: Patch coverage is 95.03106% with 8 lines in your changes missing coverage. Please review.

✅ All tests successful. No failed tests found.

Files with missing lines Patch % Lines
src/sentry/search/eap/spans.py 87.69% 8 Missing ⚠️
Additional details and impacted files
@@             Coverage Diff             @@
##           master   #83590       +/-   ##
===========================================
+ Coverage   42.10%   87.59%   +45.49%     
===========================================
  Files        9459     9485       +26     
  Lines      537327   538709     +1382     
  Branches    21221    21167       -54     
===========================================
+ Hits       226230   471905   +245675     
+ Misses     310748    66456   -244292     
+ Partials      349      348        -1     

@Zylphrex Zylphrex merged commit 73c03db into master Jan 17, 2025
49 checks passed
@Zylphrex Zylphrex deleted the txiao/feat/support-rpc-aggregate-filters branch January 17, 2025 15:08
andrewshie-sentry pushed a commit that referenced this pull request Jan 22, 2025
This adds support for aggregate filters for the table RPC.

Requires getsentry/snuba#6776.
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