[pkg/stanza] Do we need flushUncombined in the recombine operator?

[dmitryax]

There are a couple of places where we forcefully flush all the batch uncombined:

• when batching reaches MaxSources limit
• when Shutdown signal is received.

Question: Why this flush has to be uncombined? Can we do the regular flush with combined batches? It likely won't take more time, and we will not need to keep all the entries during the batching, just one (first or latest).

[github-actions]

Pinging code owners for pkg/stanza: @djaglowski. See Adding Labels via Comments if you do not have permissions to add labels yourself.

[github-actions]

Pinging code owners:

• pkg/stanza: @djaglowski

See Adding Labels via Comments if you do not have permissions to add labels yourself.

[djaglowski]

I agree we don't need it. Similar issue here, just hasn't been a priority yet. #10281

[shivanshuraj1333]

I'd like to work on it, can I get it assigned to me, thanks!

[github-actions]

This issue has been inactive for 60 days. It will be closed in 60 days if there is no activity. To ping code owners by adding a component label, see Adding Labels via Comments, or if you are unsure of which component this issue relates to, please ping @open-telemetry/collector-contrib-triagers. If this issue is still relevant, please ping the code owners or leave a comment explaining why it is still relevant. Otherwise, please close it.

Pinging code owners:

• issue: Github issue template generation code needs this to generate the corresponding labels.
• pkg/stanza: @djaglowski

See Adding Labels via Comments if you do not have permissions to add labels yourself.

[github-actions]

This issue has been inactive for 60 days. It will be closed in 60 days if there is no activity. To ping code owners by adding a component label, see Adding Labels via Comments, or if you are unsure of which component this issue relates to, please ping @open-telemetry/collector-contrib-triagers. If this issue is still relevant, please ping the code owners or leave a comment explaining why it is still relevant. Otherwise, please close it.

Pinging code owners:

• issue: Github issue template generation code needs this to generate the corresponding labels.
• pkg/stanza: @djaglowski

See Adding Labels via Comments if you do not have permissions to add labels yourself.