Skip to content

enhancement(metrics): Make transform-related functions in aggregate & tag cardinality transforms public#25358

Merged
ArunPiduguDD merged 1 commit intomasterfrom
make-aggregate-tags-cardinality-fns-pub
May 4, 2026
Merged

enhancement(metrics): Make transform-related functions in aggregate & tag cardinality transforms public#25358
ArunPiduguDD merged 1 commit intomasterfrom
make-aggregate-tags-cardinality-fns-pub

Conversation

@ArunPiduguDD
Copy link
Copy Markdown
Contributor

@ArunPiduguDD ArunPiduguDD commented May 4, 2026

Summary

Make record and flush_into in aggregate and transform_one in tag_cardinality_limit public. These will be used in follow up PRs

Change Type

  • Bug fix
  • New feature
  • Dependencies
  • Non-functional (chore, refactoring, docs)
  • Performance

Is this a breaking change?

  • Yes
  • No

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.

References

Notes

  • Please read our Vector contributor resources.
  • Do not hesitate to use @vectordotdev/vector to reach out to us regarding this PR.
  • Some CI checks run only after we manually approve them.
    • We recommend adding a pre-push hook, please see this template.
    • Alternatively, we recommend running the following locally before pushing to the remote branch:
      • make fmt
      • make check-clippy (if there are failures it's possible some of them can be fixed with make clippy-fix)
      • make test
  • After a review is requested, please avoid force pushes to help us review incrementally.
    • Feel free to push as many commits as you want. They will be squashed into one before merging.
    • For example, you can run git merge origin master and git push.
  • If this PR introduces changes Vector dependencies (modifies Cargo.lock), please
    run make build-licenses to regenerate the license inventory and commit the changes (if any). More details on the dd-rust-license-tool.

@github-actions github-actions Bot added the domain: transforms Anything related to Vector's transform components label May 4, 2026
@ArunPiduguDD ArunPiduguDD added the no-changelog Changes in this PR do not need user-facing explanations in the release changelog label May 4, 2026
@ArunPiduguDD ArunPiduguDD changed the title Make transform-related functions in aggregate & tag cardinality proce… Make transform-related functions in aggregate & tag cardinality transforms public May 4, 2026
@ArunPiduguDD ArunPiduguDD changed the title Make transform-related functions in aggregate & tag cardinality transforms public refactoring(metrics): Make transform-related functions in aggregate & tag cardinality transforms public May 4, 2026
@ArunPiduguDD ArunPiduguDD changed the title refactoring(metrics): Make transform-related functions in aggregate & tag cardinality transforms public enhancement(metrics): Make transform-related functions in aggregate & tag cardinality transforms public May 4, 2026
@ArunPiduguDD ArunPiduguDD marked this pull request as ready for review May 4, 2026 17:42
@ArunPiduguDD ArunPiduguDD requested a review from a team as a code owner May 4, 2026 17:42
@ArunPiduguDD ArunPiduguDD requested a review from pront May 4, 2026 17:42
@ArunPiduguDD ArunPiduguDD added this pull request to the merge queue May 4, 2026
Merged via the queue into master with commit 96db40e May 4, 2026
81 of 94 checks passed
@ArunPiduguDD ArunPiduguDD deleted the make-aggregate-tags-cardinality-fns-pub branch May 4, 2026 18:18
@github-actions github-actions Bot locked and limited conversation to collaborators May 4, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

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.

2 participants