| From: | David Rowley <drowley(at)postgresql(dot)org> |
|---|---|
| To: | pgsql-committers(at)lists(dot)postgresql(dot)org |
| Subject: | pgsql: Move planner row-estimation tests to new planner_est.sql |
| Date: | 2026-03-18 04:22:25 |
| Message-ID: | E1w2iQK-000GnD-0n@gemulon.postgresql.org |
| Views: | Whole Thread | Raw Message | Download mbox | Resend email |
| Thread: | |
| Lists: | pgsql-committers |
Move planner row-estimation tests to new planner_est.sql
Move explain_mask_costs() and the associated planner row-estimation
tests from misc_functions.sql to a new regression test file,
planner_est.sql.
Previously, there wasn't an ideal home for such tests, likely as there
were very few such tests due to width and selectivity estimations being
too dependent on statistics and hardware. That's not always the case, as
we have SupportRequestRows support functions. More such tests are
possibly on the way, so let's create a better home so that we don't have
to create the explain_mask_costs() function in each file we might have
added such tests to.
Author: Ilia Evdokimov <ilya(dot)evdokimov(at)tantorlabs(dot)com>
Reviewed-by: David Rowley <dgrowleyml(at)gmail(dot)com>
Discussion: https://postgr.es/m/CAApHDvphShGABn-3AoE36dTvGHW7gUpFSw0_ZZnH84wGCW3hHw@mail.gmail.com
Branch
------
master
Details
-------
https://git.postgresql.org/pg/commitdiff/374a6394c6ae2e6f8b70ca7897e1499ff72ae7e5
Modified Files
--------------
src/test/regress/expected/misc_functions.out | 178 -------------------------
src/test/regress/expected/planner_est.out | 186 +++++++++++++++++++++++++++
src/test/regress/parallel_schedule | 2 +-
src/test/regress/sql/misc_functions.sql | 125 ------------------
src/test/regress/sql/planner_est.sql | 135 +++++++++++++++++++
5 files changed, 322 insertions(+), 304 deletions(-)
| From | Date | Subject | |
|---|---|---|---|
| Next Message | Peter Eisentraut | 2026-03-18 08:11:58 | pgsql: Allow setting the collation strength in ICU tailoring rules |
| Previous Message | Michael Paquier | 2026-03-18 02:05:09 | pgsql: test_plan_advice: Add .gitignore |