On Wed, Apr 28, 2021 at 7:56 PM Tom Lane <tgl@sss.pgh.pa.us> wrote:
>
> Greg Stark <stark@mit.edu> writes:
> >> On Tue, Apr 27, 2021 at 7:08 PM Bharath Rupireddy
> >> <bharath.rupireddyforpostgres@gmail
> >> Make sense, we would lose the ability to differentiate temporary
> >> tables from the auto_explain logs.
>
> > There's no useful differentiation that can be done with the temp
> > schema name.
>
I see.
> Agreed.
>
> > I would say it makes sense to remove them -- except perhaps it makes
> > it harder to parse explain output.
>
> I don't think we should remove them. However, it could make sense to
> print the "pg_temp" alias instead of the real schema name when we
> are talking about myTempNamespace. Basically try to make that alias
> a bit less leaky.
+1, let's replace it by "pg_temp" -- did the same in that attached 0001 patch.
Also, I am wondering if we need a similar kind of handling in psql
'\d' meta-command as well? I did trial changes in the 0002 patch, but
I am not very sure about it & a bit skeptical for code change as
well. Do let me know if you have any suggestions/thoughts or if we
don't want to, so please ignore that patch, thanks.
Regards,
Amul
