Re: Clarification on Role Access Rights to Table Indexes

From: vignesh C <vignesh21(at)gmail(dot)com>
To: Nathan Bossart <nathandbossart(at)gmail(dot)com>
Cc: Ayush Vatsa <ayushvatsa1810(at)gmail(dot)com>, Robert Haas <robertmhaas(at)gmail(dot)com>, Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>, "David G(dot) Johnston" <david(dot)g(dot)johnston(at)gmail(dot)com>, PostgreSQL Hackers <pgsql-hackers(at)postgresql(dot)org>
Subject: Re: Clarification on Role Access Rights to Table Indexes
Date: 2025-03-16 13:00:23
Message-ID: CALDaNm3OD4t5z+GFhrYZooESuS3RDxNn1e9EWPiMBN7du-KGuA@mail.gmail.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-general pgsql-hackers

On Sun, 9 Mar 2025 at 03:27, Nathan Bossart <nathandbossart(at)gmail(dot)com> wrote:
>
> On Sun, Mar 09, 2025 at 03:01:41AM +0530, Ayush Vatsa wrote:
> > Maybe we can move ahead with the patch if we can see no other concerns.
>
> I think we should allow some time in case others want to review the patch.
> I do see a concern upthread about increased deadlock risk [0], but your
> patch doesn't lock the table, but unless I'm wrong [1] (which is always
> possible), it doesn't need to lock it.
>
> Anyway, here is a tidied up patch.

I noticed that Tom Lane's comment from [1] is not addressed. I'm
changing the commitfest entry status to Waiting on Author, Please
address them and update the status to Needs Review.
[1] - https://www.postgresql.org/message-id/279947.1741535285%40sss.pgh.pa.us

Regards,
Vignesh

In response to

Browse pgsql-general by date

  From Date Subject
Next Message Robert Treat 2025-03-16 22:24:59 Re: Disabling vacuum truncate for autovacuum
Previous Message Sebastien Flaesch 2025-03-16 09:30:02 Re: After upgrading libpq, the same function(PQftype) call returns a different OID

Browse pgsql-hackers by date

  From Date Subject
Next Message vignesh C 2025-03-16 13:03:07 Re: FSM doesn't recover after zeroing damaged page.
Previous Message vignesh C 2025-03-16 12:58:17 Re: Why doesn't GiST VACUUM require a super-exclusive lock, like nbtree VACUUM?