Skip to content

refactor: replace apply_fixes pipeline with plan/rewrite/repair#2489

Draft
gvozdvmozgu wants to merge 5 commits into
mainfrom
refactor/apply-fixes-plan-rewrite-repair
Draft

refactor: replace apply_fixes pipeline with plan/rewrite/repair#2489
gvozdvmozgu wants to merge 5 commits into
mainfrom
refactor/apply-fixes-plan-rewrite-repair

Conversation

@gvozdvmozgu

Copy link
Copy Markdown
Collaborator

No description provided.

@github-actions

Copy link
Copy Markdown
Contributor

Benchmark for b75eeb4

Click to view benchmark
Test Base PR %
DepthMap::from_parent 52.7±0.76µs 53.1±0.63µs +0.76%
fix_complex_query 11.7±0.32ms 9.7±0.04ms -17.09%
fix_superlong 146.9±6.64ms 149.7±4.38ms +1.91%
parse_complex_query 4.3±0.20µs 4.2±0.05µs -2.33%
parse_expression_recursion 7.1±0.36µs 7.1±0.07µs 0.00%
parse_simple_query 1052.7±27.41ns 1044.4±11.18ns -0.79%

@github-actions

Copy link
Copy Markdown
Contributor

Benchmark for 076bc82

Click to view benchmark
Test Base PR %
DepthMap::from_parent 52.8±1.82µs 53.4±1.48µs +1.14%
fix_complex_query 11.9±0.14ms 10.0±0.10ms -15.97%
fix_superlong 225.5±10.26ms 198.0±12.09ms -12.20%
parse_complex_query 4.2±0.06µs 4.3±0.05µs +2.38%
parse_expression_recursion 7.4±0.23µs 7.1±0.07µs -4.05%
parse_simple_query 1041.5±19.71ns 1043.0±13.78ns +0.14%

@github-actions

Copy link
Copy Markdown
Contributor

Benchmark for a29d7b9

Click to view benchmark
Test Base PR %
DepthMap::from_parent 52.9±0.80µs 53.3±0.77µs +0.76%
fix_complex_query 11.7±0.05ms 10.0±0.14ms -14.53%
fix_superlong 157.4±16.38ms 163.1±10.11ms +3.62%
parse_complex_query 4.1±0.05µs 4.2±0.06µs +2.44%
parse_expression_recursion 7.1±0.12µs 7.0±0.11µs -1.41%
parse_simple_query 1042.8±14.54ns 1039.1±16.12ns -0.35%

@github-actions

Copy link
Copy Markdown
Contributor

Benchmark for 18b72cf

Click to view benchmark
Test Base PR %
DepthMap::from_parent 50.4±1.37µs 48.6±2.19µs -3.57%
fix_complex_query 10.6±0.04ms 9.2±0.04ms -13.21%
fix_superlong 160.2±4.28ms 127.5±9.03ms -20.41%
parse_complex_query 4.6±0.04µs 4.4±0.04µs -4.35%
parse_expression_recursion 7.3±0.08µs 7.3±0.45µs 0.00%
parse_simple_query 973.0±11.42ns 956.8±11.15ns -1.66%

@gvozdvmozgu gvozdvmozgu force-pushed the refactor/apply-fixes-plan-rewrite-repair branch from fd6d9e9 to 62e678d Compare April 13, 2026 02:28
@github-actions

Copy link
Copy Markdown
Contributor

Benchmark for 34379ce

Click to view benchmark
Test Base PR %
DepthMap::from_parent 62.1±0.50µs 61.9±1.06µs -0.32%
fix_complex_query 13.0±0.09ms 11.0±0.33ms -15.38%
fix_superlong 188.8±8.68ms 154.6±16.43ms -18.11%
parse_complex_query 4.4±0.05µs 4.4±0.05µs 0.00%
parse_expression_recursion 7.6±0.10µs 7.5±0.07µs -1.32%
parse_simple_query 1054.9±12.80ns 1076.0±29.73ns +2.00%

@github-actions

github-actions Bot commented May 3, 2026

Copy link
Copy Markdown
Contributor

Benchmark for a26e395

Click to view benchmark
Test Base PR %
DepthMap::from_parent 61.3±0.43µs 61.1±0.73µs -0.33%
fix_complex_query 12.8±0.08ms 10.9±0.05ms -14.84%
fix_superlong 205.6±7.48ms 165.4±12.98ms -19.55%
parse_complex_query 4.4±0.05µs 4.3±0.09µs -2.27%
parse_expression_recursion 7.6±0.10µs 7.6±0.10µs 0.00%
parse_simple_query 1108.6±18.84ns 1072.9±25.53ns -3.22%

@codspeed-hq

codspeed-hq Bot commented May 3, 2026

Copy link
Copy Markdown

Merging this PR will improve performance by 12.78%

⚡ 2 improved benchmarks
✅ 4 untouched benchmarks
⏩ 6 skipped benchmarks1

Performance Changes

Mode Benchmark BASE HEAD Efficiency
WallTime fix_complex_query 36.6 ms 32.4 ms +12.78%
WallTime fix_superlong 492.7 ms 445.8 ms +10.52%

Comparing refactor/apply-fixes-plan-rewrite-repair (966dd92) with main (f20b401)

Open in CodSpeed

Footnotes

  1. 6 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.

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