Repository navigation
chore: Remove integration-specific Python queue ops - #19900
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub.
1 Skipped Deployment
|
|
Please update the IS YOUR CHANGE URGENT? section of the PR description:
This information helps the Docs team prioritize your review. |
|
The plan checks client errors, pageload failure, and navigation failure. An issue escalates when those error or failure rates rise. Telemetry cannot prove the removed table rows are gone. Services: PlanWhat changedThe Span Operations page no longer lists
RiskA broken MDX table can fail render on the shared Intended effectThis change is a docs table edit. No counter records that those three ops left the page. Do not treat a green error rate as proof the rows are gone. Regression watchTraces sample at 30%. Compare like for like. Hourly rates swing on small samples. Use the 24h rate as the primary band. Existing Algolia 401 and hydration issues are noise unless they spike on the Span Operations URL.
Not observableThe published table text is not in telemetry. The executor cannot confirm those three ops are gone from HTML. Sampled pageloads for that URL were 0 in the 24h window, so short-window watches on that path are empty. |
| | queue | | | | ||
| | | queue.task | | | ||
| | | queue.task.celery | | | ||
| | | queue.task.rq | | | ||
| | | queue.task.delayed_job | | | ||
| | | queue.task.active_job | | | ||
| | | queue.submit | | | ||
| | | queue.submit.celery | | | ||
| | | queue.resque | | | ||
| | | queue.sidekiq | | |
There was a problem hiding this comment.
This doesn't seem to align with conventions (task/submit vs publish/process). Is this a mistake or am I missing something?
There was a problem hiding this comment.
conventions are the source of truth.
not sure why it doesn't align.
|
Verified in
|
DESCRIBE YOUR PR
Related to getsentry/sentry-conventions#693
IS YOUR CHANGE URGENT?
Help us prioritize incoming PRs by letting us know when the change needs to go live.
Select exactly one option. For deadlines, replace
YYYY-MM-DDwith the due date. You can update this information later by editing the PR description.SLA
Thanks in advance for your help!
PRE-MERGE CHECKLIST
Make sure you've checked the following before merging your changes:
LEGAL BOILERPLATE
Look, I get it. The entity doing business as "Sentry" was incorporated in the State of Delaware in 2015 as Functional Software, Inc. and is gonna need some rights from me in order to utilize my contributions in this here PR. So here's the deal: I retain all rights, title and interest in and to my contributions, and by keeping this boilerplate intact I confirm that Sentry can use, modify, copy, and redistribute my contributions, under Sentry's choice of terms.
EXTRA RESOURCES