Feedback-ID: ie3de48e3:Fastmail
Date: Thu, 13 Nov 2025 09:24:52 +0100
From: =?utf-8?Q?=C3=81lvaro?= Herrera <alvherre@kurilemu.de>
To: jian he <jian.universality@gmail.com>
Cc: Peter Eisentraut <peter@eisentraut.org>, Tom Lane <tgl@sss.pgh.pa.us>,
	PostgreSQL-development <pgsql-hackers@postgresql.org>,
 Kirill Reshke <reshkekirill@gmail.com>
Subject: Re: misleading error message in ProcessUtilitySlow T_CreateStatsStmt
Message-ID: <202511130823.vppi3clyjl4r@alvherre.pgsql>
MIME-Version: 1.0
Content-Type: text/plain; charset=utf-8
Content-Disposition: inline
Content-Transfer-Encoding: 8bit
In-Reply-To: 
 <CACJufxHJ4xXM_svKE3y3=cdsKuM1Ke4PXY0K=xZcCmWw4sjwBQ@mail.gmail.com>
Archived-At: 
 <https://www.postgresql.org/message-id/202511130823.vppi3clyjl4r%40alvherre.pgsql>
Precedence: bulk

On 2025-Nov-13, jian he wrote:

> @@ -15658,10 +15658,19 @@ ATPostAlterTypeParse(Oid oldId, Oid oldRelId, Oid refRelId, char *cmd,
>  			querytree_list = list_concat(querytree_list, afterStmts);
>  		}
>  		else if (IsA(stmt, CreateStatsStmt))
> +		{
> +			RangeTblEntry 	*rte;
> +			CreateStatsStmt *ss = castNode(CreateStatsStmt, stmt);
> +
> +			rte = makeNode(RangeTblEntry);
> +			rte->rtekind = RTE_RELATION;
> +			rte->relid = oldRelId;
> +			rte->rellockmode = ShareUpdateExclusiveLock;
> +			ss->rtable = list_make1(rte);
> +
>  			querytree_list = lappend(querytree_list,
> -									 transformStatsStmt(oldRelId,
> -														(CreateStatsStmt *) stmt,
> -														cmd));
> +									 transformStatsStmt(ss, cmd));
> +		}
>  		else
>  			querytree_list = lappend(querytree_list, stmt);
>  	}

Hmm, how would this part here work in the hypothetical world where a
stats object references multiple relations?

-- 
Álvaro Herrera        Breisgau, Deutschland  —  https://www.EnterpriseDB.com/
"No renuncies a nada. No te aferres a nada."