Remove duplicate query logging that breaks Django's assertNumQueries

The SqlCommenter _QueryWrapper manually appends the SQL string to
queries_log when a CursorDebugWrapper is active. However,
CursorDebugWrapper.debug_sql already logs the query as a dict with
"sql" and "time" keys. This causes two problems:

1. Queries are double-counted in queries_log
2. The raw string entries cause a TypeError in Django's
   assertNumQueries, which expects dict entries with a "sql" key

Remove the manual append and let Django handle query logging.

Author: federicobond

instrumentation/opentelemetry-instrumentation-django/src/opentelemetry/instrumentation/django/middleware/sqlcommenter_middleware.py

@@ -18,7 +18,6 @@
 # pylint: disable=no-name-in-module
 from django import conf, get_version
 from django.db import connections
-from django.db.backends.utils import CursorDebugWrapper
 
 from opentelemetry.instrumentation.sqlcommenter_utils import _add_sql_comment
 from opentelemetry.instrumentation.utils import _get_opentelemetry_values
@@ -122,8 +121,4 @@ def __call__(self, execute: Type[T], sql, params, many, context) -> T:
         #  * https://github.com/basecamp/marginalia/issues/61
         #  * https://github.com/basecamp/marginalia/pull/80
 
-        # Add the query to the query log if debugging.
-        if isinstance(context["cursor"], CursorDebugWrapper):
-            context["connection"].queries_log.append(sql)
-
         return execute(sql, params, many, context)