Skip to content

Conversation

@hengfeiyang
Copy link
Contributor

@hengfeiyang hengfeiyang commented Nov 27, 2025

fixed #9330
update vrl to v0.28.1

@github-actions
Copy link
Contributor

Failed to generate code suggestions for PR

@greptile-apps
Copy link
Contributor

greptile-apps bot commented Nov 27, 2025

Greptile Overview

Greptile Summary

This PR updates the VRL (Vector Remap Language) library from v0.26.0 to v0.28.1 and the vector-enrichment dependency from branch v0.49.0-24092025 to v0.51. The update adds the arbitrary feature to VRL, which is commonly used for property-based testing and fuzzing support.

Key Changes:

  • VRL version bump: v0.26.0v0.28.1 (minor version updates, backward compatible)
  • Added arbitrary feature to VRL dependency for enhanced testing capabilities
  • Updated vector-enrichment branch: v0.49.0-24092025v0.51
  • Transitive dependency updates: dyn-clone (1.0.19→1.0.20), new dependencies instant and ipcrypt-rs
  • Multiple parking_lot version references updated across dependent crates

The changes are straightforward dependency updates without any code modifications. VRL is used throughout the codebase for data transformation functions, enrichment tables, and JWT handling. The version update appears safe as it follows semantic versioning and no breaking API changes are expected between minor versions.

Confidence Score: 5/5

  • This PR is safe to merge with minimal risk - it's a routine dependency update following semantic versioning
  • Score reflects that this is a minor version update (v0.26.0→v0.28.1) following semantic versioning principles, which guarantees backward compatibility. No code changes were made, only dependency updates in Cargo files. The codebase uses VRL extensively for transformations and enrichment, but the VRL API surface used (compiler, runtime, functions) remains stable across minor versions. The added arbitrary feature is optional and used for testing/fuzzing, not production code.
  • No files require special attention

Important Files Changed

File Analysis

Filename Score Overview
Cargo.toml 5/5 Updated VRL from v0.26.0 to v0.28.1, added arbitrary feature, and updated vector-enrichment branch from v0.49.0-24092025 to v0.51
Cargo.lock 5/5 Updated dependency versions including dyn-clone (1.0.19→1.0.20), added new dependencies (instant, ipcrypt-rs), and updated parking_lot references across multiple crates

Sequence Diagram

sequenceDiagram
    participant Dev as Developer
    participant CT as Cargo.toml
    participant CL as Cargo.lock
    participant VRL as VRL Library
    participant VE as Vector Enrichment
    
    Dev->>CT: Update vrl version to 0.28.1
    Dev->>CT: Add "arbitrary" feature
    Dev->>CT: Update vector-enrichment branch to v0.51
    CT->>CL: Trigger dependency resolution
    CL->>VRL: Resolve vrl v0.28.1 with features
    CL->>VE: Resolve vector-enrichment from v0.51 branch
    VRL->>CL: Pull transitive dependencies (dyn-clone, instant, ipcrypt-rs)
    CL->>CL: Update parking_lot references across crates
    CL-->>Dev: Dependency tree updated
Loading

Copy link
Contributor

@greptile-apps greptile-apps bot left a comment

Choose a reason for hiding this comment

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

1 file reviewed, no comments

Edit Code Review Agent Settings | Greptile

@hengfeiyang hengfeiyang merged commit 2c67254 into main Nov 27, 2025
46 of 50 checks passed
@hengfeiyang hengfeiyang deleted the ci/vrl branch November 27, 2025 23:50
@hengfeiyang hengfeiyang added the Needs-Testing Needs-Testing label Nov 28, 2025
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.

Vrl函数parse_aws_alb_log需要更新

4 participants