Skip to content

chore: moved parallel_build_large test from regression tests to rust tests#4041

Merged
mdashti merged 3 commits intomainfrom
moe/remove-long-running-regression
Feb 2, 2026
Merged

chore: moved parallel_build_large test from regression tests to rust tests#4041
mdashti merged 3 commits intomainfrom
moe/remove-long-running-regression

Conversation

@mdashti
Copy link
Copy Markdown
Contributor

@mdashti mdashti commented Jan 31, 2026

Ticket(s) Closed

  • Closes #N/A

What

Converted the parallel_build_large regression test to Rust #[pg_test] tests.

Why

The test creates 35,000 rows and runs 32 configuration combinations, making it very slow for regular local runs. Moving it to Rust keeps the test available for CI runs without slowing down the standard dev regression test suite.

How

  • Added two new tests to build_parallel.rs:
    • test_parallel_build_large_insufficient_memory - verifies low-memory failure
    • test_parallel_build_large_configurations - tests all worker/memory/segment combinations
  • Removed the pg_regress SQL files

@mdashti mdashti added the Do Not Cherry Pick PR should not be cherry-picked to other branches label Feb 2, 2026
@mdashti mdashti merged commit 8b8dff1 into main Feb 2, 2026
26 of 28 checks passed
@mdashti mdashti deleted the moe/remove-long-running-regression branch February 2, 2026 08:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Do Not Cherry Pick PR should not be cherry-picked to other branches

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants