From 97ce821e3e171ce99fa7c398889ac08432cd0264 Mon Sep 17 00:00:00 2001 From: Alexander Korotkov Date: Wed, 3 Apr 2024 21:29:18 +0300 Subject: [PATCH] Fix the parameters order for TableAmRoutine.relation_copy_for_cluster() Specify OldTable first, NewTable second as used by table_relation_copy_for_cluster() and as implemented in heapam_relation_copy_for_cluster(). Backpatch to PostgreSQL 12, where TableAmRoutine was introduced. Discussion: https://postgr.es/m/ME3P282MB3166860D4911AE82F92DF7C5B63F2%40ME3P282MB3166.AUSP282.PROD.OUTLOOK.COM Author: Japin Li Reviewed-by: Pavel Borisov Backpatch-through: 12 --- src/include/access/tableam.h | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/src/include/access/tableam.h b/src/include/access/tableam.h index 2c1a540155a..e7eeb754098 100644 --- a/src/include/access/tableam.h +++ b/src/include/access/tableam.h @@ -631,8 +631,8 @@ typedef struct TableAmRoutine const RelFileLocator *newrlocator); /* See table_relation_copy_for_cluster() */ - void (*relation_copy_for_cluster) (Relation NewTable, - Relation OldTable, + void (*relation_copy_for_cluster) (Relation OldTable, + Relation NewTable, Relation OldIndex, bool use_sort, TransactionId OldestXmin, -- 2.30.2