Skip to content

chore: better tracing (generator.GenerateSchema, WriteSchema)#2981

Open
miparnisari wants to merge 1 commit intomainfrom
add-ctx-to-GenerateSchema
Open

chore: better tracing (generator.GenerateSchema, WriteSchema)#2981
miparnisari wants to merge 1 commit intomainfrom
add-ctx-to-GenerateSchema

Conversation

@miparnisari
Copy link
Contributor

@miparnisari miparnisari commented Mar 21, 2026

Description

The goal of this PR is to avoid having separate root traces for generator.GenerateSchemaWithTypeSet on CheckPermission calls.

Before:

image

After:

image

@miparnisari miparnisari requested a review from a team as a code owner March 21, 2026 00:00
@github-actions github-actions bot added area/schema Affects the Schema Language area/api v1 Affects the v1 API area/tooling Affects the dev or user toolchain (e.g. tests, ci, build tools) labels Mar 21, 2026
@codecov
Copy link

codecov bot commented Mar 21, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 74.80%. Comparing base (70fe885) to head (dd4d1b6).

❌ Your project check has failed because the head coverage (74.80%) is below the target coverage (75.00%). You can increase the head coverage or adjust the target coverage.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #2981      +/-   ##
==========================================
- Coverage   74.85%   74.80%   -0.05%     
==========================================
  Files         499      499              
  Lines       61063    61072       +9     
==========================================
- Hits        45703    45677      -26     
- Misses      12174    12197      +23     
- Partials     3186     3198      +12     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@miparnisari miparnisari force-pushed the add-ctx-to-GenerateSchema branch from ececa4a to 99ffc8a Compare March 21, 2026 00:12
@miparnisari miparnisari enabled auto-merge (squash) March 21, 2026 00:51
@miparnisari miparnisari force-pushed the add-ctx-to-GenerateSchema branch from 99ffc8a to dd4d1b6 Compare March 21, 2026 01:12
@miparnisari miparnisari changed the title chore: pass ctx to generator.GenerateSchema chore: better tracing (generator.GenerateSchema, WriteSchema) Mar 21, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area/api v1 Affects the v1 API area/schema Affects the Schema Language area/tooling Affects the dev or user toolchain (e.g. tests, ci, build tools) Skip-Changelog

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant