Re: [PATCH] Tab complete EXECUTE FUNCTION for CREATE (EVENT) TRIGGER
Dagfinn Ilmari Mannsåker <ilmari@ilmari.org>
From: ilmari@ilmari.org (Dagfinn Ilmari Mannsåker )
To: Tom Lane <tgl@sss.pgh.pa.us>
Cc: Michael Paquier <michael@paquier.xyz>, David Fetter <david@fetter.org>,
pgsql-hackers@postgresql.org
Date: 2018-10-26T13:40:29Z
Lists: pgsql-hackers
Tom Lane <tgl@sss.pgh.pa.us> writes: > Michael Paquier <michael@paquier.xyz> writes: >> On Fri, Oct 26, 2018 at 09:31:48AM +0900, Michael Paquier wrote: >>> Committed 0001 now which adds tab completion for CREATE TRIGGER. > >> And 0002 is committed as well. Thanks for the patches! > > The committed patches look sane to me, but should we back-patch into 11? > This isn't quite a bug fix maybe, but it's inconsistent if v11 server & > docs recommend this syntax while v11 psql doesn't produce it. A related issue is whether we should change pg_get_triggerdef() to emit the new syntax as well (in HEAD only, since it would break things that parse the output). - ilmari -- "I use RMS as a guide in the same way that a boat captain would use a lighthouse. It's good to know where it is, but you generally don't want to find yourself in the same spot." - Tollef Fog Heen
Commits
-
Add tab completion of EXECUTE FUNCTION for CREATE TRIGGER in psql
- fdf5eade9626 11.1 landed
- 292ef6e2778a 12.0 landed
-
Improve tab completion of CREATE EVENT TRIGGER in psql
- 5953c9969762 12.0 landed
-
Improve tab completion for ANALYZE, EXPLAIN, and VACUUM.
- 121213d9d852 12.0 cited