|From:||Michael Paquier <michael(at)paquier(dot)xyz>|
|To:||Masahiko Sawada <sawada(dot)mshk(at)gmail(dot)com>|
|Cc:||Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>, Alexey Bashtanov <bashtanov(at)imap(dot)cc>, pgsql-bugs <pgsql-bugs(at)postgresql(dot)org>, Mi Tar <mmitar(at)gmail(dot)com>|
|Subject:||Re: Is temporary functions feature official/supported? Found some issues with it.|
|Views:||Raw Message | Whole Thread | Download mbox | Resend email|
On Tue, Jan 15, 2019 at 03:36:56PM +0900, Masahiko Sawada wrote:
> Thank you for updating the patch. The patch looks good to me. And the
> new regression test for CREATE EXTENSION seems to work fine but maybe
> it's better to reset client_min_messages at cleanup for safety.
Sure, done. I have been working on this patch more this morning, and
here is a proposal for commit. So please let me know if there are any
objections with that.
9b013dc is the commit which has introduced MyXactFlags, so this means
that we cannot get that back-patched further down than v10. Per the
lack of complains, that's a restriction I can live with.
The first patch is the actual fix to back-patch. The second patch is
an improvement I propose only for HEAD which removes ACCESSEDTEMPREL,
replacing it with ACCESSEDTEMPNAMESPACE. For now I propose to commit
0001, then I'll spawn a new thread to discuss 0002 on -hackers as
XACT_FLAGS_ACCESSEDTEMPREL has the advantage to allow skipping ON
COMMIT DELETE if no temporary tables have been accessed. Perhaps
that's not worth bothering, but that point seems worth poking at, at
least to me.
|Next Message||Amit Langote||2019-01-16 09:30:02||Re: BUG #15587: Partitions with ALTER TABLE ADD CONSTRAINT|
|Previous Message||David Rowley||2019-01-15 23:38:55||Re: BUG #15572: Misleading message reported by "Drop function operation" on DB with functions having same name|