Add GraphQL OpenTelemetry integration#1395
Draft
shangyian wants to merge 3129 commits intoDataJunction:mainfrom
Draft
Add GraphQL OpenTelemetry integration#1395shangyian wants to merge 3129 commits intoDataJunction:mainfrom
shangyian wants to merge 3129 commits intoDataJunction:mainfrom
Conversation
GQL common dimensions query + dim attr output shape
… provide grain information
Only generate preaggregated measures SQL where possible
Add measures SQL query to GQL
…as the roads demo
…tion#1388) * Fix the derived expression of count distinct metrics to use the count distinct argument * Add separate treatment of measures for distinct quantifier aggregations * Add support to complex expressions inside distinct-agg metrics * Refactor names and add tests for resolving metric components * Cleanup * More naming cleanup * Add graphql naming refactor * Update comments and update GraphQL schema
* Add pagination to /cubes endpoint * don't pull inactive nodes for /cubes/... endpoints
* Add endpoint to list available dialects * Add support for GraphQL querying of dialects
✅ Deploy Preview for thriving-cassata-78ae72 canceled.
|
Co-authored-by: GitHub Actions Bot <>
* Fix an issue where renaming dimensions for measures SQL creates should include both columns * Fix tests * Refactor to make it cleaner
…1396) * Add option to delete materializations by node version. * Lint and commments addressed. * Comment addressed plus added node_revision_id to Materialization output.
* Replaces use of Depends(get_session) in get_context() with session middleware, since the former doesn't support teardown in Strawberry. * Add test for db session middleware * Fix tests
…ataJunction#1402) * When a node is deactivated, its materializations should be as well * Fix tests
Co-authored-by: GitHub Actions Bot <>
* Add filtering on columns for node revisions in GQL * Add tests for resolvers
* Add docker compose with superset * Remove comments * add password init * Add secret key * docker compose file that works with superset * Fix * Fix names * Fix --------- Co-authored-by: Beto Dealmeida <roberto@dealmeida.net>
Co-authored-by: GitHub Actions Bot <>
Fix the ignore files for unit test workflow.
* Fix test action. * Add UI code filter.
* Skip deactivated nodes in namespace export. * Adjust unit test.
* Add support for reader clusters * Add ability to switch between reader and writer cluster across REST and GraphQL calls * Fix tests * Add test for reader/writer session * Add example .env * Fix * Fix tests * Set database connection max overflow to 100 * Clean up * Fix graphql tests * Fix * Fix * Fix
Co-authored-by: GitHub Actions Bot <>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
6 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Adds GraphQL OpenTelemetry integration for tracing. See https://strawberry.rocks/docs/extensions/opentelemetry
Test Plan
make checkpassesmake testshows 100% unit test coverageDeployment Plan