From: | Shayon Mukherjee <shayonj(at)gmail(dot)com> |
---|---|
To: | Sami Imseih <samimseih(at)gmail(dot)com> |
Cc: | Gurjeet Singh <gurjeet(at)singh(dot)im>, David Rowley <dgrowleyml(at)gmail(dot)com>, Nathan Bossart <nathandbossart(at)gmail(dot)com>, pgsql-hackers(at)lists(dot)postgresql(dot)org |
Subject: | Re: [PATCH] Re: Proposal to Enable/Disable Index using ALTER INDEX |
Date: | 2025-04-05 21:00:59 |
Message-ID: | CANqtF-rKPYnMj3x0BVXQyY8k18jtBrvwvOTkYAvK_bYVvcpaOQ@mail.gmail.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
On Wed, Apr 2, 2025 at 10:53 PM Sami Imseih <samimseih(at)gmail(dot)com> wrote:
> > That seems like a very good location for this advice. But the current
> > set of bullet points are all directed towards "... a general procedure
> > for determining which indexes to create". I propose that a new paragrph,
> > not a bullet point, be added towards the end of that section which
> > addresses the options of steps to take before dropping an index.
> > Something like the following:
>
> > Thoughts?
>
> This new feature provides the ability to experiment with indexes to
> create ( or drop ),
> so I don't think it deviates from the purpose of this paragraphs.
>
> This patch will provide the ability for the user to create an index as
> initially
> invisible and a GUC, use_invisible_index if set to TRUE to experiment with
> the new index to see if it improves performance. So, I think providing this
> pattern to experiment with a new index will fit nicely as a new
> bulletpoint.
>
>
>
Thank you for the feedback and pointers Sami and Gurjeet. Good call on [0]
being a good place for operational advice. I have gone ahead and removed
the advice about "pg_stat_user_indexes.idx_scan"
from doc/src/sgml/ref/alter_index.sgml and
updated doc/src/sgml/indices.sgml to include a new bullet point with also a
reference to use_invisible_index. Let me know how it sounds and if there is
any feedback.
Also, rebased.
Thank you
Shayon
[0] https://www.postgresql.org/docs/current/indexes-examine.html
Attachment | Content-Type | Size |
---|---|---|
v15-0001-Introduce-the-ability-to-set-index-visibility-us.patch | application/octet-stream | 85.5 KB |
From | Date | Subject | |
---|---|---|---|
Next Message | Alena Rybakina | 2025-04-05 21:01:57 | Re: Removing unneeded self joins |
Previous Message | Tom Lane | 2025-04-05 20:37:08 | Re: CREATE OR REPLACE MATERIALIZED VIEW |