Skip to content

feat(spans): Support recent searches#72938

Merged
Zylphrex merged 1 commit into
masterfrom
txiao/feat/support-recent-search-for-spans
Jun 18, 2024
Merged

feat(spans): Support recent searches#72938
Zylphrex merged 1 commit into
masterfrom
txiao/feat/support-recent-search-for-spans

Conversation

@Zylphrex

Copy link
Copy Markdown
Member

We were using the event search type for trace search. Creating a span type so we can switch over.

We were using the event search type for trace search. Creating a span type so we
can switch over.
@Zylphrex Zylphrex requested a review from a team June 18, 2024 15:46
@github-actions github-actions Bot added the Scope: Backend Automatically applied to PRs that change backend components label Jun 18, 2024
Zylphrex added a commit that referenced this pull request Jun 18, 2024
This was using the event type so searches from other parts of the product like
discover were being mixed here. Switch to use it's own type.

Depends on #72938
@codecov

codecov Bot commented Jun 18, 2024

Copy link
Copy Markdown

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 78.05%. Comparing base (29ff72e) to head (2fd5c07).
Report is 40 commits behind head on master.

Additional details and impacted files
@@            Coverage Diff             @@
##           master   #72938      +/-   ##
==========================================
+ Coverage   73.27%   78.05%   +4.77%     
==========================================
  Files        6602     6606       +4     
  Lines      294280   294505     +225     
  Branches    50731    50764      +33     
==========================================
+ Hits       215645   229881   +14236     
+ Misses      72049    58294   -13755     
+ Partials     6586     6330     -256     
Files Coverage Δ
src/sentry/models/search_common.py 100.00% <100.00%> (ø)

... and 870 files with indirect coverage changes

@Zylphrex Zylphrex merged commit 8c790ab into master Jun 18, 2024
@Zylphrex Zylphrex deleted the txiao/feat/support-recent-search-for-spans branch June 18, 2024 17:01
Zylphrex added a commit that referenced this pull request Jun 18, 2024
This was using the event type so searches from other parts of the
product like discover were being mixed here. Switch to use it's own
type.

Depends on #72938
@github-actions github-actions Bot locked and limited conversation to collaborators Jul 4, 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.

2 participants