On 2026-Sep-23, Álvaro Herrera wrote:

> On 2026-Sep-23, Chao Li wrote:

> > But I have concern with the change. cluster_rel() is used by both
> > REPACK and CLUSTER commands, but CLUSTER command, the syntax is
> > CLUSTER … USING index-name.
> 
> Yeah, I think we should keep the "cannot execute %s on a shared catalog"
> string as is, and make the SQL bit interpolated there be REPACK USING
> INDEX or CLUSTER according to stmt->cmd; one because CLUSTER also allows
> the older CLUSTER ON syntax AFAIR, but also you can't do CLUSTER with no
> index.

Pushed like this, thanks.

-- 
Álvaro Herrera         PostgreSQL Developer  —  https://www.EnterpriseDB.com/
"I suspect most samba developers are already technically insane...
Of course, since many of them are Australians, you can't tell." (L. Torvalds)


Reply via email to