vectordotdev/vector

Refactor field iterators to return `OwnedTargetPath`s

Open

#21,077 opened on Aug 14, 2024

View on GitHub
 (8 comments) (0 reactions) (0 assignees)Rust (2,126 forks)batch import
domain: coregood first issuetype: tech debt

Repository metrics

Stars
 (21,837 stars)
PR merge metrics
 (Avg merge 5d 12h) (85 merged PRs in 30d)

Description

These functions live here:

This should eliminate unnecessary path editing like this.

Contributor guide

Research direction
The issue points to two files: `lib/vector core/src/event/log event.rs` and `lib/vector core/src/event/util/log/all fields.rs`. The goal is to refactor the field iterators in these files to return `OwnedTargetPath` instead of the current implementation, which will eliminate unnecessary path editing. Look at the existing code, especially the linked line in `all fields.rs` where path editing occurs, and understand how `OwnedTargetPath` is used elsewhere in the codebase.
Tech stack
rust
Domain
backenddata
Issue type
Refactor
DifficultyEstimated implementation difficulty for a new contributor, from 1 for very small changes to 5 for expert-level work.
3
Estimated timeA rough time range for an experienced contributor to investigate, implement, test, and prepare a pull request.
1-3 hours
Activity statusHow available the issue appears right now: fresh, active, stale, blocked, or waiting on maintainer input.
Active
ClarityHow clearly the issue explains the expected change, acceptance criteria, and next step.
Mostly clear
Prerequisites
Basic Rust knowledgeFamiliarity with vector's event system
Newbie friendlinessA 1-100 score estimating how approachable this issue is for first-time contributors.
45