Logflare / logflare / 5215a47d582be875070b32af3095efec33199335

81%

push

github

Follow-ups to #3458: schema_utils + bench profiling (#3465)

* refactor(schema): pattern-match do_flatten_typemap join_key

Splits the empty-prefix check out of the `if`/`else` inside the
`Enum.reduce` callback. Follow-up to feedback on #3458.

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

* test(schema): consolidate flatten_typemap leaf tests, add property test

Per #3458 feedback: the empty/single/multi/datetime/list-typed top-level
cases were five separate tests with overlapping setup. Collapse into one
case-based test and add a StreamData property that compares
`flatten_typemap/1` against an oracle implementation across random
typemap shapes.

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

* chore(bench): extract snapshot/delta logic into Logflare.Profiling

Per #3458 feedback: the inline snapshot/history/delta-print block in
`log_event_make_bench.exs` was a candidate for reuse across other bench
scripts under `test/profiling/`. Move it into `Logflare.Profiling` so
each script reduces to `Benchee.run(...) |> Profiling.track(history_path)`.
Pure refactor — same behavior, no new fields surfaced.

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

* feat(bench): show ips delta column

Per #3458 review feedback (Ziinc): wall-time alone is harder to reason
about than throughput. Add an ips row to `Profiling.print_delta/2`'s
output. The history file already captured ips per scenario, so this is
display-only — no snapshot format change.

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

* chore(bench): fold tprof into Benchee profile_after, drop separate script

Per #3458 feedback (ruslandoga): Benchee 1.1+ ships a `profile_after`
option that wraps :tprof (and the other built-in profilers) against the
same scenarios used for timing. Add `PROFILE=1` + `TPROF_TYPE` env knobs
that route into Benchee instead of maintaining
`test/profiling/log_event_make_profile.exs` as a pa... (continued)

4 of 4 new or added lines in 2 files covered. (100.0%)

12479 of 15608 relevant lines covered (79.95%)

10535.93 hits per line