Skip to content

Conversation

@philipphofmann
Copy link
Member

@philipphofmann philipphofmann commented Apr 9, 2025

This PR migrates the SpansAggregator RFC to the develop docs. As logs also have to use some aggregation, we renamed the SpansAggregator to BatchProcessor so it can be used with any type of telemetry data.

This PR migrates the SpansAggregator RFC to the develop docs. As logs
also have to use some aggregation, we renamed the SpansAggregator to
BatchProcessor so it can be used with any type of telemetry data.
@vercel
Copy link

vercel bot commented Apr 9, 2025

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
develop-docs ✅ Ready (Inspect) Visit Preview 💬 Add feedback Apr 10, 2025 0:34am
2 Skipped Deployments
Name Status Preview Comments Updated (UTC)
changelog ⬜️ Ignored (Inspect) Visit Preview Apr 10, 2025 0:34am
sentry-docs ⬜️ Ignored (Inspect) Visit Preview Apr 10, 2025 0:34am

Copy link
Member

@AbhiPrasad AbhiPrasad left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice! Really happy with where we're going with this.

Now that we have this, I feel like these are the next extension items. We can put them into an issue and tackle them one at a time though.

  1. figuring out if we need priority queue in the processor
  2. optimizing for trace completeness (related to 1)
  3. detailing behaviour during crashes

@philipphofmann philipphofmann merged commit f19ebbb into master Apr 11, 2025
10 checks passed
@philipphofmann philipphofmann deleted the feat/batch-processor branch April 11, 2025 12:46
bitsandfoxes pushed a commit that referenced this pull request Apr 24, 2025
This PR migrates the SpansAggregator RFC to the develop docs. As logs
also have to use some aggregation, we renamed the SpansAggregator to
BatchProcessor so it can be used with any type of telemetry data.
@github-actions github-actions bot locked and limited conversation to collaborators Apr 27, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants