From 72e51d3cbd17529440205bfb4a56889c2aa3d6dc Mon Sep 17 00:00:00 2001 From: Tony Hsu Date: Tue, 23 Jan 2024 12:43:23 +0100 Subject: [PATCH] Deprecate tracer option --- guides/queries/tracing.md | 8 +------- lib/graphql/tracing/data_dog_trace.rb | 1 + lib/graphql/tracing/data_dog_tracing.rb | 1 + 3 files changed, 3 insertions(+), 7 deletions(-) diff --git a/guides/queries/tracing.md b/guides/queries/tracing.md index 8d34573e0c..2b1a239228 100644 --- a/guides/queries/tracing.md +++ b/guides/queries/tracing.md @@ -148,16 +148,10 @@ To add [Datadog](https://www.datadoghq.com) instrumentation: ```ruby class MySchema < GraphQL::Schema - use(GraphQL::Tracing::DataDogTracing, options) + use(GraphQL::Tracing::DataDogTracing) end ``` -You may provide `options` as a `Hash` with the following values: - -| Key | Description | Default | -| --- | ----------- | ------- | -| `tracer` | Tracer used to perform instrumentation. Usually you don't need to set this. | `Datadog::Tracing` | - For more details about Datadog's tracing API, check out the [Ruby documentation](https://github.com/DataDog/dd-trace-rb/blob/master/docs/GettingStarted.md) or the [APM documentation](https://docs.datadoghq.com/tracing/) for more product information. ## Prometheus diff --git a/lib/graphql/tracing/data_dog_trace.rb b/lib/graphql/tracing/data_dog_trace.rb index 93c36722ce..33c08850a3 100644 --- a/lib/graphql/tracing/data_dog_trace.rb +++ b/lib/graphql/tracing/data_dog_trace.rb @@ -3,6 +3,7 @@ module GraphQL module Tracing module DataDogTrace + # @param tracer [#trace] Deprecated # @param analytics_enabled [Boolean] Deprecated # @param analytics_sample_rate [Float] Deprecated def initialize(tracer: nil, analytics_enabled: false, analytics_sample_rate: 1.0, service: nil, **rest) diff --git a/lib/graphql/tracing/data_dog_tracing.rb b/lib/graphql/tracing/data_dog_tracing.rb index 035531c650..ef3d2fc975 100644 --- a/lib/graphql/tracing/data_dog_tracing.rb +++ b/lib/graphql/tracing/data_dog_tracing.rb @@ -56,6 +56,7 @@ def prepare_span(key, data, span) def tracer default_tracer = defined?(Datadog::Tracing) ? Datadog::Tracing : Datadog.tracer + # [Deprecated] options[:tracer] will be removed in the future options.fetch(:tracer, default_tracer) end