Re: [PATCH] Remove make_temptable_name_n() - Mailing list pgsql-hackers

From Nathan Bossart
Subject Re: [PATCH] Remove make_temptable_name_n()
Date
Msg-id aOkqVUninoLnpOpD@nathan
Whole thread Raw
In response to [PATCH] Remove make_temptable_name_n()  (Aleksander Alekseev <aleksander@tigerdata.com>)
List pgsql-hackers
On Fri, Oct 10, 2025 at 06:22:46PM +0300, Aleksander Alekseev wrote:
> The proposed patch removes make_temptable_name_n() in matview.c. This
> function is used only once and can be replaced with psprintf().

This dates back to commit cc1965a, and I see some discussion about this
function in the corresponding thread [0].  One thing I don't like about
the patch is that we lose the comment about relying on the name to never be
double-quoted.  IMHO that's worth retaining in some form.

[0] https://postgr.es/m/CAP7Qgm%3Djb3xkzQXfGtX9STx8fzd8EDDQ-oJ8ekcyeOud%2ByLCoA%40mail.gmail.com

-- 
nathan



pgsql-hackers by date:

Previous
From: Aleksander Alekseev
Date:
Subject: [PATCH] Remove make_temptable_name_n()
Next
From: Bertrand Drouvot
Date:
Subject: Re: Preserve index stats during ALTER TABLE ... TYPE ...