Skip to content

fix: use empty url to use the same url as the playground#21

Merged
StarpTech merged 3 commits intomainfrom
dustin/allow-to-graphql-url
Aug 29, 2023
Merged

fix: use empty url to use the same url as the playground#21
StarpTech merged 3 commits intomainfrom
dustin/allow-to-graphql-url

Conversation

@StarpTech
Copy link
Copy Markdown
Contributor

Motivation and Context

@StarpTech StarpTech merged commit 0eaa1a3 into main Aug 29, 2023
@ysmolski ysmolski deleted the dustin/allow-to-graphql-url branch September 1, 2025 14:24
jensneuse added a commit that referenced this pull request Apr 24, 2026
Entity cache metrics were created whenever entity caching was enabled,
ignoring the opt-in pattern used by connection_stats. Mirror the
connection_stats convention:

- Add entity_caching_stats bool (default false) to telemetry.metrics.otlp
  and telemetry.metrics.prometheus in config.schema.json
- Matching Go struct fields with env bindings METRICS_OTLP_ENTITY_CACHING_STATS
  and PROMETHEUS_ENTITY_CACHING_STATS
- Propagate through MetricConfigFromTelemetry into rmetric.Config
- Gate OTLPEntityCache / PromEntityCache collector creation in
  setupEntityCacheMetrics on the per-backend flag
- Parallel coverage in config_test.go, config_defaults.json, config_full.json,
  fixtures/full.yaml, and graph_server_cache_metrics_test.go

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants