Skip to content

chore(deps): bump VRL for query performance improvements - #26649

Merged
jackie8c merged 3 commits into
masterfrom
yoenn.burban/bump-vrl
Oct 13, 2026
Merged

jackie8c merged 3 commits into
masterfrom
yoenn.burban/bump-vrl

Conversation

@gwenaskell

Copy link
Copy Markdown
Contributor

Summary

Bring the recent VRL query performance improvements into Vector by updating the locked revision from 09f1cd85 to deef5c37.

  • Read variable subpaths without cloning the whole variable.
  • Reuse cached paths when reading event and metadata fields.
  • Adapt set_semantic_meaning to the borrowed Query::external_path() API.
  • Replace deprecated Value::as_str() calls with its equivalent, to_str_lossy().

References

Vector configuration

Used tests/behavior/transforms/remap.yaml to derive the feature set for Clippy.

How did you test this PR?

  • Passed make check-clippy with FEATURES=transforms-remap,vector-vrl-functions/dnstap,vrl-functions-crypto,vrl-functions-env,vrl-functions-network,vrl-functions-system.
  • Passed make fmt, make check-fmt, and make check-markdown.
  • Ran make build-licenses; retained existing license entries since the bump adds no packages and regeneration only removed unrelated entries.
  • Tests left to CI, as requested. The local test command stopped before running tests because cargo-nextest is not installed.

Does this PR include user facing changes?

  • Yes. Added a performance changelog fragment.
  • No. A maintainer will apply the no-changelog label to this PR.

Contributor Guidelines

Followed the repository contribution guidance. PR opened as a draft.

@gwenaskell
gwenaskell marked this pull request as ready for review October 13, 2026 14:31
@gwenaskell
gwenaskell requested a review from a team as a code owner October 13, 2026 14:31
@chatgpt-codex-connector

chatgpt-codex-connector Bot commented Oct 13, 2026 •

Copy link
Copy Markdown

Codex Review Summary

This comment shows the latest Codex review activity on this pull request.

Review Status Commit Review trigger
📝 Code Review ✅ Completed 2026-10-13T18:24:53.097317Z fd0ef87 New commits
ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review" or "@codex security review".

Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings.

@datadoghq-integration datadoghq-integration Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Bits Code Review: FAIL

The VRL bump introduces deprecation errors in Kubernetes log source and Splunk HEC sink builds because their remaining Value::as_str() calls conflict with Vector’s deny-warnings policy.

Open Bits AI session

🤖 Bits Code Review · Commit b98028a

Comment thread Cargo.lock
@github-actions github-actions Bot added domain: topology Anything related to Vector's topology code domain: sources Anything related to Vector's sources domain: sinks Anything related to Vector's sinks labels Oct 13, 2026
@gwenaskell
gwenaskell enabled auto-merge October 13, 2026 15:56
@gwenaskell
gwenaskell added this pull request to the merge queue Oct 13, 2026
@github-merge-queue
github-merge-queue Bot removed this pull request from the merge queue due to a conflict with the base branch Oct 13, 2026
@github-actions github-actions Bot removed domain: topology Anything related to Vector's topology code domain: sources Anything related to Vector's sources domain: sinks Anything related to Vector's sinks labels Oct 13, 2026
@github-actions

github-actions Bot commented Oct 13, 2026 •

Copy link
Copy Markdown
Contributor

All contributors have signed the CLA ✍️ ✅
Posted by the CLA Assistant Lite bot.

@jackie8c
jackie8c enabled auto-merge October 13, 2026 18:21
@jackie8c

Copy link
Copy Markdown
Contributor

I have read the CLA Document and I hereby sign the CLA

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: fd0ef87cdc

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment thread Cargo.lock
@jackie8c
jackie8c disabled auto-merge October 13, 2026 18:32
@jackie8c
jackie8c enabled auto-merge October 13, 2026 18:42
@jackie8c
jackie8c added this pull request to the merge queue Oct 13, 2026
Merged via the queue into master with commit 1144931 Oct 13, 2026
108 of 109 checks passed
@jackie8c
jackie8c deleted the yoenn.burban/bump-vrl branch October 13, 2026 19:20
@github-actions github-actions Bot locked and limited conversation to collaborators Oct 13, 2026
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.

3 participants