Re: REPACK (CONCURRENTLY) fails with wrong error for materialized views

From: Álvaro Herrera <alvherre(at)kurilemu(dot)de>
To: Bharath Rupireddy <bharath(dot)rupireddyforpostgres(at)gmail(dot)com>
Cc: PostgreSQL-development <pgsql-hackers(at)postgresql(dot)org>, Nathan Bossart <nathandbossart(at)gmail(dot)com>, Kiran Kaki <itskkpg(at)gmail(dot)com>
Subject: Re: REPACK (CONCURRENTLY) fails with wrong error for materialized views
Date: 2026-09-09 11:02:44
Message-ID: aqE8AMclUokBJvib@alvherre.pgsql
Views: Whole Thread | Raw Message | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

On 2026-Aug-31, Bharath Rupireddy wrote:

> Thoughts?

Hello, thanks, I have pushed this. I removed the CREATE INDEX from the
test case though, because it's not needed: the new error fires before
even trying to find the index. I also moved to the start of the list
the documentation addition: the new sentence was the only "the relation
is ..." in the midst of a bunch of "the table is ..." statements, so it
felt out of place. And I made it "the relation is not a table" instead
(with a parenthical comment mentioning a matview), so that all the
others "the table is ..." make sense as followups to it.

Thanks!

--
Álvaro Herrera 48°01'N 7°57'E — https://www.EnterpriseDB.com/
Subversion to GIT: the shortest path to happiness I've ever heard of
(Alexey Klyukin)

In response to

Browse pgsql-hackers by date

  From Date Subject
Next Message Greg Burd 2026-09-09 11:14:01 Re: Speed up COPY FROM text/CSV parsing using SIMD
Previous Message Rafia Sabih 2026-09-09 11:00:19 Re: Add statistics refresh materialized view