On Tue, Sep 08, 2020 at 09:02:38PM -0300, Alvaro Herrera wrote: > On 2020-Sep-08, Justin Pryzby wrote: > > > From 992e0121925c74d5c5a4e5b132cddb3d6b31da86 Mon Sep 17 00:00:00 2001 > > From: Justin Pryzby <pryz...@telsasoft.com> > > Date: Fri, 27 Mar 2020 17:50:46 -0500 > > Subject: [PATCH v27 1/5] Change REINDEX/CLUSTER to accept an option list.. > > > > ..like EXPLAIN (..), VACUUM (..), and ANALYZE (..). > > > > Change docs in the style of VACUUM. See also: > > 52dcfda48778d16683c64ca4372299a099a15b96 > > I don't understand why you change all options to DefElem instead of > keeping the bitmask for those options that can use it.
That's originally how I did it, too. Initially I added List *params, and Michael suggested to retire ReindexStmt->concurrent. I provided a patch to do so, initially by leaving int options and then, after this, removing it to "complete the thought", and get rid of the remnants of the "old way" of doing it. This is also how vacuum and explain are done. https://www.postgresql.org/message-id/20200902022410.GA20149%40telsasoft.com -- Justin
