From 38585549a3ef96fbbd8a1e7c86dc1b4b8ee5faa8 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 29f67d5153..cca3f64980 100644 --- a/src/include/access/tableam.h +++ b/src/include/access/tableam.h @@ -471,8 +471,8 @@ typedef struct TableAmRoutine const RelFileNode *newrnode); /* 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,