Skip to content

6% perf gain: Don't write fields to strings preemptively#71

Merged
9999years merged 1 commit intomainfrom
push-oqroswtqplky
Sep 6, 2025
Merged

6% perf gain: Don't write fields to strings preemptively#71
9999years merged 1 commit intomainfrom
push-oqroswtqplky

Conversation

@9999years
Copy link
Owner

This seems to improve performance by 5.7%!

format event/colors, wrapping, stderr
    time:   [11.696 µs 11.714 µs 11.737 µs]
    change: [-6.2772% -5.7379% -5.1220%] (p = 0.00 < 0.05)
format event/colors, wrapping
    time:   [5.2662 µs 5.2711 µs 5.2747 µs]
    change: [-14.121% -13.943% -13.777%] (p = 0.00 < 0.05)
format event/no colors, wrapping
    time:   [5.1020 µs 5.1102 µs 5.1164 µs]
    change: [-10.160% -10.017% -9.8873%] (p = 0.00 < 0.05)
format event/colors, no wrapping
    time:   [1.7943 µs 1.7968 µs 1.8006 µs]
    change: [-25.048% -24.892% -24.716%] (p = 0.00 < 0.05)
format event/no colors, no wrapping
    time:   [1.4897 µs 1.4916 µs 1.4947 µs]
    change: [-22.072% -21.960% -21.832%] (p = 0.00 < 0.05)

This seems to improve performance by 5.7%!

format event/colors, wrapping, stderr
    time:   [11.696 µs 11.714 µs 11.737 µs]
    change: [-6.2772% -5.7379% -5.1220%] (p = 0.00 < 0.05)
format event/colors, wrapping
    time:   [5.2662 µs 5.2711 µs 5.2747 µs]
    change: [-14.121% -13.943% -13.777%] (p = 0.00 < 0.05)
format event/no colors, wrapping
    time:   [5.1020 µs 5.1102 µs 5.1164 µs]
    change: [-10.160% -10.017% -9.8873%] (p = 0.00 < 0.05)
format event/colors, no wrapping
    time:   [1.7943 µs 1.7968 µs 1.8006 µs]
    change: [-25.048% -24.892% -24.716%] (p = 0.00 < 0.05)
format event/no colors, no wrapping
    time:   [1.4897 µs 1.4916 µs 1.4947 µs]
    change: [-22.072% -21.960% -21.832%] (p = 0.00 < 0.05)
@9999years 9999years merged commit e808324 into main Sep 6, 2025
32 checks passed
@9999years 9999years deleted the push-oqroswtqplky branch September 6, 2025 00:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant