pgsql: Remove variable "concurrent" from ReindexStmt

From: Michael Paquier <michael(at)paquier(dot)xyz>
To: pgsql-committers(at)lists(dot)postgresql(dot)org
Subject: pgsql: Remove variable "concurrent" from ReindexStmt
Date: 2020-09-04 01:51:18
Message-ID: E1kE0so-00043J-7g@gemulon.postgresql.org
Views: Whole Thread | Raw Message | Download mbox | Resend email
Thread:
Lists: pgsql-committers

Remove variable "concurrent" from ReindexStmt

This node already handles multiple options using a bitmask, so having a
separate boolean flag is not necessary. This simplifies the code a bit
with less arguments to give to the reindex routines, by replacing the
boolean with an equivalent bitmask value.

Reviewed-by: Julien Rouhaud
Discussion: https://postgr.es/m/20200902110326.GA14963@paquier.xyz

Branch
------
master

Details
-------
https://git.postgresql.org/pg/commitdiff/844c05abc3f1c1703bf17cf44ab66351ed9711d2

Modified Files
--------------
src/backend/commands/indexcmds.c | 33 ++++++++++++++++++++-------------
src/backend/nodes/copyfuncs.c | 1 -
src/backend/nodes/equalfuncs.c | 1 -
src/backend/parser/gram.y | 12 ++++++++----
src/backend/tcop/utility.c | 8 ++++----
src/include/commands/defrem.h | 6 +++---
src/include/nodes/parsenodes.h | 2 +-
7 files changed, 36 insertions(+), 27 deletions(-)

Browse pgsql-committers by date

  From Date Subject
Next Message Bruce Momjian 2020-09-04 02:57:46 pgsql: remove redundant initializations
Previous Message Tom Lane 2020-09-04 00:09:40 pgsql: Remove arbitrary restrictions on password length.