Re: trivial grammar refactor - Mailing list pgsql-hackers

From Álvaro Herrera
Subject Re: trivial grammar refactor
Date
Msg-id 202507240954.uage7twscx4s@alvherre.pgsql
Whole thread Raw
In response to Re: trivial grammar refactor  (Andres Freund <andres@anarazel.de>)
Responses Re: trivial grammar refactor
List pgsql-hackers
Hello,

On 2025-Jul-23, Andres Freund wrote:

> On 2025-07-23 19:59:52 +0200, Álvaro Herrera wrote:
> > ... so using the same set of productions, I can rewrite the current
> > CLUSTER rule in this way and it won't be a problem for the REPACK
> > changes.
> 
> But it comes at the price of henceforth duplicating all ClusterStmt, once for
> VERBOSE and once without. That's not exactly a free lunch...

Yeah, thanks for taking a look.  That duplication is just me being dumb.
Here's a version without that.  The only thing that needed to change was
changing "CLUSTER opt_verbose" to "CLUSTER VERBOSE" so that the
unadorned CLUSTER is handled by "CLUSTER opt_utility_option_list"
instead.


-- 
Álvaro Herrera               48°01'N 7°57'E  —  https://www.EnterpriseDB.com/
"How amazing is that? I call it a night and come back to find that a bug has
been identified and patched while I sleep."                (Robert Davidson)
               http://archives.postgresql.org/pgsql-sql/2006-03/msg00378.php

Attachment

pgsql-hackers by date:

Previous
From: shveta malik
Date:
Subject: Re: Logical replication launcher did not automatically restart when got SIGKILL
Next
From: Nazir Bilal Yavuz
Date:
Subject: Re: Improve error reporting in 027_stream_regress test