From: | Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> |
---|---|
To: | pgsql-committers(at)postgresql(dot)org |
Subject: | pgsql: More code review for rangetypes patch. |
Date: | 2011-11-21 21:20:13 |
Message-ID: | E1RSbHh-0003dl-45@gemulon.postgresql.org |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-committers |
More code review for rangetypes patch.
Fix up some infelicitous coding in DefineRange, and add some missing error
checks. Rearrange operator strategy number assignments for GiST anyrange
opclass so that they don't make such a mess of opr_sanity's table of
operator names associated with different strategy numbers. Assign
hopefully-temporary selectivity estimators to range operators that didn't
have one --- poor as the estimates are, they're still a lot better than the
default 0.5 estimate, and they'll shut up the opr_sanity test that wants to
see selectivity estimators on all built-in operators.
Branch
------
master
Details
-------
http://git.postgresql.org/pg/commitdiff/a4ffcc8e115ed637f69ecb0295d78cc97f08a483
Modified Files
--------------
src/backend/commands/typecmds.c | 395 ++++++++++++++++--------------
src/backend/utils/adt/rangetypes_gist.c | 113 +++++-----
src/include/catalog/catversion.h | 2 +-
src/include/catalog/pg_amop.h | 22 +-
src/include/catalog/pg_operator.h | 12 +-
src/test/regress/expected/opr_sanity.out | 50 ++---
src/test/regress/expected/plpgsql.out | 14 -
src/test/regress/expected/rangetypes.out | 14 +
src/test/regress/sql/opr_sanity.sql | 20 +-
src/test/regress/sql/plpgsql.sql | 10 -
src/test/regress/sql/rangetypes.sql | 7 +
11 files changed, 342 insertions(+), 317 deletions(-)
From | Date | Subject | |
---|---|---|---|
Next Message | Tom Lane | 2011-11-22 02:32:44 | Re: [COMMITTERS] pgsql: Avoid marking buffer dirty when VACUUM has no work to do. |
Previous Message | Bruce Momjian | 2011-11-21 20:08:33 | Re: pgsql: Do missed autoheader run for previous commit. |