Skip to content

refactor(go-forwarder): batcher take channels as parameters#1104

Merged
ndakkoune merged 1 commit intonabil.dakkoune/go-forwarderfrom
nabil.dakkoune/refactor-batching
Apr 13, 2026
Merged

refactor(go-forwarder): batcher take channels as parameters#1104
ndakkoune merged 1 commit intonabil.dakkoune/go-forwarderfrom
nabil.dakkoune/refactor-batching

Conversation

@ndakkoune
Copy link
Copy Markdown
Contributor

What does this PR do?

Refactors the Batcher to make the main pipeline consistent

Motivation

Testing Guidelines

Additional Notes

Types of changes

  • Bug fix
  • New feature
  • Breaking change
  • Misc (docs, refactoring, dependency upgrade, etc.)

Check all that apply

  • This PR's description is comprehensive
  • This PR contains breaking changes that are documented in the description
  • This PR introduces new APIs or parameters that are documented and unlikely to change in the foreseeable future
  • This PR impacts documentation, and it has been updated (or a ticket has been logged)
  • This PR's changes are covered by the automated tests
  • This PR collects user input/sensitive content into Datadog
  • This PR passes the integration tests (ask a Datadog member to run the tests)
  • This PR passes the unit tests
  • This PR passes the installation tests (ask a Datadog member to run the tests)

@github-actions github-actions bot added the aws label Apr 10, 2026
@ndakkoune ndakkoune changed the title refactor(go-forwarder): take channels as parameters refactor(go-forwarder): batcher take channels as parameters Apr 10, 2026
@ndakkoune ndakkoune marked this pull request as ready for review April 10, 2026 15:13
@ndakkoune ndakkoune requested a review from a team as a code owner April 10, 2026 15:13
@ViBiOh ViBiOh self-assigned this Apr 13, 2026
@ndakkoune ndakkoune merged commit 690d812 into nabil.dakkoune/go-forwarder Apr 13, 2026
10 checks passed
@ndakkoune ndakkoune deleted the nabil.dakkoune/refactor-batching branch April 13, 2026 08:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants