Skip to content

feat(aggregate): expose final flush - #26437

Draft
ArunPiduguDD wants to merge 1 commit into
masterfrom
arun.pidugu/expose-aggregate-final-flush
Draft

ArunPiduguDD wants to merge 1 commit into
masterfrom
arun.pidugu/expose-aggregate-final-flush

Conversation

@ArunPiduguDD

Copy link
Copy Markdown
Contributor

Summary

Expose Aggregate::flush_final publicly so wrappers can drain open event-time buckets during shutdown without constructing and driving a temporary transform stream.

References

Related: event-time aggregation support

Vector configuration

N/A — this only exposes an existing aggregate transform method.

How did you test this PR?

  • cargo fmt --all -- --check
  • make test FEATURES="transforms-aggregate" SCOPE="aggregate" (48 passed)
  • FEATURES="transforms-aggregate" make check-clippy reaches an unrelated pre-existing clippy::unused_async failure in src/test_util/http.rs

Does this PR include user facing changes?

  • Yes. Please add a changelog fragment based on our guidelines.
  • No. A maintainer will apply the no-changelog label to this PR.

Contributor Guidelines

  • Please read our Vector contributor resources.
  • Do not hesitate to use @vectordotdev/vector to reach out to us regarding this PR.
  • Before pushing, follow our pre-push guidance.
  • After a review is requested, please avoid force pushes to help us review incrementally.

Allow aggregate wrappers to drain open event-time buckets directly during shutdown.

Environment: Datadog workspace

Co-Authored-By: Pi OpenAI GPT-5.6 <noreply@pi.dev>
@ArunPiduguDD ArunPiduguDD added the no-changelog Changes in this PR do not need user-facing explanations in the release changelog label Sep 18, 2026 — with ddtool CLI
@github-actions github-actions Bot added the domain: transforms Anything related to Vector's transform components label Sep 18, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

domain: transforms Anything related to Vector's transform components no-changelog Changes in this PR do not need user-facing explanations in the release changelog

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant