pgsql: Remove IndexInfo.ii_OpclassOptions field

From: Peter Eisentraut <peter(at)eisentraut(dot)org>
To: pgsql-committers(at)lists(dot)postgresql(dot)org
Subject: pgsql: Remove IndexInfo.ii_OpclassOptions field
Date: 2023-10-03 15:55:04
Message-ID: E1qnhjk-007K76-HC@gemulon.postgresql.org
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-committers

Remove IndexInfo.ii_OpclassOptions field

It is unnecessary to include this field in IndexInfo. It is only used
by DDL code, not during execution. It is really only used to pass
local information around between functions in index.c and indexcmds.c,
for which it is clearer to use local variables, like in similar cases.

Discussion: https://www.postgresql.org/message-id/flat/f84640e3-00d3-5abd-3f41-e6a19d33c40b(at)eisentraut(dot)org

Branch
------
master

Details
-------
https://git.postgresql.org/pg/commitdiff/784162357130f63b5130cd6517db21451692f9b3

Modified Files
--------------
src/backend/catalog/index.c | 33 +++++++++++++++------------------
src/backend/catalog/toasting.c | 3 +--
src/backend/commands/indexcmds.c | 30 +++++++++++++++++-------------
src/backend/nodes/makefuncs.c | 3 ---
src/include/catalog/index.h | 1 +
src/include/nodes/execnodes.h | 1 -
6 files changed, 34 insertions(+), 37 deletions(-)

Browse pgsql-committers by date

  From Date Subject
Next Message Tom Lane 2023-10-03 18:14:01 pgsql: Doc: suppress "exceed the available area" warning in PDF build.
Previous Message Tom Lane 2023-10-03 15:41:51 pgsql: Add some notes about why "ALTER TYPE enum DROP VALUE" is hard.