Skip to content

Remove needless fmt_sql #7409

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Draft
gatesn wants to merge 1 commit into
base: develop
Choose a base branch
from

Remove fmt_sql

d0a444e
Select commit
Loading
Failed to load commit list.
Draft

Remove needless fmt_sql #7409

Remove fmt_sql
d0a444e
Select commit
Loading
Failed to load commit list.
CodSpeed HQ / CodSpeed Performance Analysis succeeded Apr 13, 2026 in 0s

Performance Gate Passed

⚡ 2 improved benchmarks
✅ 1121 untouched benchmarks
⏩ 1455 skipped benchmarks1

Performance Changes

Mode Benchmark BASE HEAD Efficiency
⚡ Simulation varbinview_zip_fragmented_mask 7.1 ms 6.4 ms +11.73%
⚡ Simulation varbinview_zip_block_mask 3.7 ms 2.9 ms +27.63%

Comparing ngates/remove-default-fmt-sql (d0a444e) with develop (d4e7dca)

Open in CodSpeed

Footnotes

  1. 1455 benchmarks were skipped, so the baseline results were used instead. If they were deleted from the codebase, click here and archive them to remove them from the performance reports. ↩

View more details on CodSpeed HQ