From: Tom Lane <tgl@sss.pgh.pa.us>
To: Fujii Masao <masao.fujii@oss.nttdata.com>
cc: Noah Misch <noah@leadboat.com>, Nathan Bossart <nathandbossart@gmail.com>,
        vaibhave postgres <postgresvaibhave@gmail.com>,
        pgsql-bugs@lists.postgresql.org, vsekar@microsoft.com
Subject: Re: vacuumdb: permission denied for schema "pg_temp_7"
In-reply-to: <5f37ee56-14fb-44fe-9150-9eb97e10538b@oss.nttdata.com>
References: 
 <CAM_eQjwfAR=y3G1fGyS1U9FTmc+FyJm9amNfY2QCZBnDDbNPZg@mail.gmail.com>
 <20240920190731.37.nmisch@google.com> <Zu3iMzfiGBTbg3iy@nathan>
 <20240920215030.8d.nmisch@google.com> <2603142.1726873660@sss.pgh.pa.us>
 <5f37ee56-14fb-44fe-9150-9eb97e10538b@oss.nttdata.com>
Comments: In-reply-to Fujii Masao <masao.fujii@oss.nttdata.com>
	message dated "Sat, 21 Sep 2024 14:59:51 +0900"
MIME-Version: 1.0
Content-Type: text/plain; charset="us-ascii"
Content-ID: <2649164.1726898595.1@sss.pgh.pa.us>
Content-Transfer-Encoding: quoted-printable
Date: Sat, 21 Sep 2024 02:03:15 -0400
Message-ID: <2649165.1726898595@sss.pgh.pa.us>
Archived-At: 
 <https://www.postgresql.org/message-id/2649165.1726898595%40sss.pgh.pa.us>
Precedence: bulk

Fujii Masao <masao.fujii@oss.nttdata.com> writes:
> It looks like reindexdb has the same issue. It would be good to
> update reindexdb to skip temp tables as well to fix this.

Agreed.

> For the proposed patch, it seems better to use CppAsString2(RELPERSISTEN=
CE_TEMP)
> instead of 't'.

+1, if we can easily avoid hard-coding that value we should do so.
It's not that we're going to change the value; it's that it makes
it way easier to grep the source tree for relevant code.

			regards, tom lane