Logflare / logflare / 3511d21223cce3c9c8938101c8a6ba60083acc6f-PR-3449

80%

main: 80%

Build Type

Pull #3449

github

Committed by djwhitt

Commit Message

perf: replace put_in/Access.key with direct put_at_path recursion

copy_fields and apply_kv_instruction were doing
`put_in(acc, Enum.map(to_path, &Access.key(&1, %{})), value)` on every
event, which allocates one closure per path segment plus an intermediate
list. The new put_at_path/3 recurses directly over the parsed string
list (mirroring drop_field_at/2), eliminating those allocations on the
ingest hot path.

Benchmark (3s/2s, MIX_ENV=test): copy_fields parsed median drops from
~50μs to ~45μs per event (~10%) with no behavior change.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>

Pull Request Pull Request #3449: feat: add drop_fields ingest transform

Coverage Stats

38 of 39 new or added lines in 3 files covered. (97.44%)

1 existing line in 1 file now uncovered.

12372 of 15552 relevant lines covered (79.55%)

4968.32 hits per line