fix(graphql): rename service entry span (#4053) (#4055)

The graphql top level span is currently named `graphql.query`. This span wraps the execution of the `graphql.graphql()`. The current span name can be misleading since `graphql.graphql()` can execute graphql queries, graphql mutations or graphql subscriptions. This fix proposes setting the span name to `graphql.request` instead.

An alternative fix would be to set the span name to  `graphql.query`, `graphql.mutation` or `graphql.subscription` depending on the operation name. However getting the operation name would require parsing the query and this can be expensive.


Co-authored-by: Munir Abdinur <[email protected]>
Co-authored-by: mergify[bot] <37929162+mergify[bot]@users.noreply.github.com>

Author: mergify[bot]

ddtrace/contrib/graphql/patch.py

@@ -182,7 +182,7 @@ def _traced_query(func, args, kwargs):
     resource = _get_source_str(source)
 
     with pin.tracer.trace(
-        name="graphql.query",
+        name="graphql.request",
         resource=resource,
         service=trace_utils.int_service(pin, config.graphql),
         span_type=SpanTypes.GRAPHQL,

tests/snapshots/tests.contrib.graphql.test_graphql.test_graphql.json

@@ -1,6 +1,6 @@
 [[
   {
-    "name": "graphql.query",
+    "name": "graphql.request",
     "service": "graphql",
     "resource": "query HELLO { hello }",
     "trace_id": 0,

tests/snapshots/tests.contrib.graphql.test_graphql.test_graphql_error.json

@@ -1,6 +1,6 @@
 [[
   {
-    "name": "graphql.query",
+    "name": "graphql.request",
     "service": "graphql",
     "resource": "{ invalid_schema }",
     "trace_id": 0,

tests/snapshots/tests.contrib.graphql.test_graphql.test_graphql_v2_with_document.json

@@ -24,7 +24,7 @@
   }],
 [
   {
-    "name": "graphql.query",
+    "name": "graphql.request",
     "service": "graphql",
     "resource": "query HELLO { hello }",
     "trace_id": 1,

tests/snapshots/tests.contrib.graphql.test_graphql.test_graphql_with_traced_resolver.json

@@ -1,6 +1,6 @@
 [[
   {
-    "name": "graphql.query",
+    "name": "graphql.request",
     "service": "graphql",
     "resource": "query HELLO { hello }",
     "trace_id": 0,