Re: CREATE MATERIALIZED VIEW
Dagfinn Ilmari Mannsåker <ilmari@ilmari.org>
From: Dagfinn Ilmari Mannsåker <ilmari@ilmari.org>
To: Nathan Bossart <nathandbossart@gmail.com>
Cc: px shi <spxlyy123@gmail.com>, pgsql-hackers@postgresql.org
Date: 2024-07-25T15:10:37Z
Lists: pgsql-hackers
Nathan Bossart <nathandbossart@gmail.com> writes: > On Thu, Jul 25, 2024 at 03:49:02PM +0100, Dagfinn Ilmari Mannsåker wrote: >> Nathan Bossart <nathandbossart@gmail.com> writes: >>> However, since it looks like the feature was never actually supported in a >>> release, and the revert has been in place for over a decade, I think it'd >>> be reasonable to remove the tab completion now. It looks like the folks on >>> the 2021 thread felt similarly. >> >> Keeping it in the grammar makes sense for the more specific error >> message, but I don't think the tab completion should be suggesting bogus >> things, regardless of whether it's the grammar or the parse analysis >> that rejects it. > > Would you mind creating a commitfest entry for this one? I'll plan on > committing this early next week unless any objections materialize. Done: https://commitfest.postgresql.org/49/5139/ I've taken the liberty of setting you as the committer, and the target version to 17 even though it turns out to be an older bug, since it's arguably a follow-on fix to the incomplete fix in c951e9042dd1. - ilmari
Commits
-
Remove tab completion for CREATE UNLOGGED MATERIALIZED VIEW.
- 5c1ce1bbbe5f 18.0 landed
-
Add tab completion for CREATE SCHEMA in psql
- c951e9042dd1 17.0 cited
-
Disallow unlogged materialized views.
- 3223b25ff737 9.3.0 cited
-
Add a materialized view relations.
- 3bf3ab8c5636 9.3.0 cited