Skip to content

Conversation

@sentrivana
Copy link
Contributor

@sentrivana sentrivana commented Nov 19, 2025

Description

WIP: Draft implementation of span first/span streaming in the Python SDK. Uses the existing start_span API, but batches spans and sends them independently of the transaction/segment span.

Enables to run the SDK in two modes: static and streaming. Static mode is the current status quo where spans are sent together with their transaction (segment) as the transaction finishes. Streaming mode makes two major changes to the behavior of the start_span API:

  • Spans are added to the streaming buffer on finish and they may be sent in an envelope without their segment.
  • [Later] start_span without a running segment will be promoted to a segment span. (For the POC, this is not the case; we just use the existing start_transaction.)

TODOs:

  • ignore_spans
  • start_span should start a segment span if there is no active segment span

Issues

Reminders

@codecov
Copy link

codecov bot commented Nov 19, 2025

❌ 1 Tests Failed:

Tests completed Failed Passed Skipped
28666 1 28665 2318
View the top 1 failed test(s) by shortest run time
tests.integrations.aws_lambda.test_aws_lambda::test_error_has_existing_trace_context[RaiseErrorPerformanceDisabled]
Stack Traces | 5.03s run time
.../integrations/aws_lambda/test_aws_lambda.py:85: in test_environment
    process.wait(timeout=5)  # Give it time to shut down gracefully
    ^^^^^^^^^^^^^^^^^^^^^^^
.../hostedtoolcache/Python/3.13.9....../x64/lib/python3.13/subprocess.py:1280: in wait
    return self._wait(timeout=timeout)
           ^^^^^^^^^^^^^^^^^^^^^^^^^^^
.../hostedtoolcache/Python/3.13.9....../x64/lib/python3.13/subprocess.py:2058: in _wait
    raise TimeoutExpired(self.args, timeout)
E   subprocess.TimeoutExpired: Command '['sam', 'local', 'start-lambda', '--debug', '--template', 'sam.template.yaml', '--warm-containers', 'EAGER', '--docker-network', 'lambda-test-network']' timed out after 5 seconds

To view more test analytics, go to the Test Analytics Dashboard
📋 Got 3 mins? Take this short survey to help us improve Test Analytics.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants