At line 104, the type annotation declares span: "Optional[Span]" but the code at line 107 explicitly handles StreamedSpan instances via isinstance(span, StreamedSpan). This inconsistency could cause static type checkers to flag the isinstance check as always-false, and it misrepresents the actual runtime behavior where _sentry_sql_span can be either Span or StreamedSpan.

Also found at:

• sentry_sdk/tracing_utils.py:178

A print(span) statement on line 486 appears to be debug code accidentally left in. This will pollute test output by printing every span object during test execution, making test logs noisy and harder to read.

The span_streaming path in test_transactions is missing the SPANDATA.SERVER_ADDRESS not in span["attributes"] assertion that exists in the non-streaming path (line 203). The non-streaming path validates both SERVER_ADDRESS and SERVER_PORT are absent from span data, but the streaming path only validates SERVER_PORT. This creates a gap in test coverage where the streaming implementation could incorrectly include SERVER_ADDRESS without test failure.

Also found at:

• tests/integrations/sqlalchemy/test_sqlalchemy.py:289-293