public inbox for [email protected]
help / color / mirror / Atom feedFrom: Chao Li <[email protected]>
To: Greg Sabino Mullane <[email protected]>
Cc: Jim Jones <[email protected]>
Cc: David G. Johnston <[email protected]>
Cc: Postgres hackers <[email protected]>
Subject: Re: ALTER TABLE: warn when actions do not recurse to partitions
Date: Wed, 11 Mar 2026 15:04:34 +0800
Message-ID: <[email protected]> (raw)
In-Reply-To: <CAKAnmmKzzOWKQVhaAt8LrBmUW=FLWBqS4v8cwKOG53hBToQZRA@mail.gmail.com>
References: <CAEoWx2=SLga-xH09Cq_PAvsHhQHrBK+V0vF821JKgzS=Bm0haA@mail.gmail.com>
<CAKFQuwZXt2dAuHFs6MtRCzfMP6YZFzssuzdW2woJBVmco=CDdQ@mail.gmail.com>
<CAEoWx2nic2MaPUEKwb0Me1iwO3LsCf8wUHm55pjqaTRgJEZpcQ@mail.gmail.com>
<CAKFQuwahxpPKT-LXGJ34BO5cVBUxFxzgNPTY8E_VW-cuNU_DkQ@mail.gmail.com>
<CAEoWx2kQ9aSn-0LGPKc=woj+h-HR4uBV9TL20aeM3HzLmFqgFQ@mail.gmail.com>
<[email protected]>
<[email protected]>
<CAEoWx2mZsWnyS_XLDL5mL+yuU_g65p_tmHNesyfngqow7gtS3A@mail.gmail.com>
<[email protected]>
<[email protected]>
<[email protected]>
<[email protected]>
<[email protected]>
<[email protected]>
<[email protected]>
<CAKAnmmKzzOWKQVhaAt8LrBmUW=FLWBqS4v8cwKOG53hBToQZRA@mail.gmail.com>
> On Mar 10, 2026, at 23:32, Greg Sabino Mullane <[email protected]> wrote:
>
> Review of v6:
Thank you very much for the review.
>
> typedef struct partitionNoRecurseNotice
> {
> List *notices;
> } partitionNoRecurseNotice;
> Not sure why we need a struct here, rather than just passing the list around?
Initially I thought there might be a few fields in the structure, but ended up only one List field. Yes, this structure is not needed now. Removed it in v7.
>
> Also should be PartitionNoRecurseNotice (CamelCase)
>
> foreach(cell, postNotice->notices)
> {
> if (strcmp((char *) lfirst(cell), notice_msg) == 0)
> {
> pfree(notice_msg);
> found = true;
> break;
> }
> }
>
> This seems a lot of extra work that could be avoided. Since we know each message is unique to the cmdtype/AlterTableType and the rel/Relation combination, use those two to drive the duplicate check. Then we can only build the notice_msg if needed! Perhaps adding two more fields to that lonely struct above?
Are you concerning that rendering the full message text is the extra work? This is not a hot path, so I don’t think that would be a big deal. Actually, adding two more fields sounds more expensive.
>
> partitionNoRecurseNotice * postNotice);
>
> postNotice is a little misleading - maybe pending_notices or just notices?
Yes, “pending” makes sense. Updated in v7.
>
> does not affect present partitions
>
> s/present/existing/g
> CollectPartitionNoRecurseNotice(AT_SetSchema, rel, stmt->relation->inh, false, &postNotice);
>
> This hard-coded AT_SetSchema just to return a "SET SCHEMA" later on feels hacky. Don't have a workaround off the top of my head, just registering my mild unease. :)
Yes, as SET SCHEMA doesn’t go through the standard ALTER TABLE process: AlterTable() -> ATController() -> ATPrepCmd().
If you get some idea, please let me know.
>
> /* Emit a notice only if there are partitions */
> if (nparts == 0)
> return;
>
> It doesn't look like this particular case is tested. Other than that, the tests look very good.
Good catch. Added a test case to cover that.
PFA v7: addressed Greg’s review comments.
Best regards,
--
Chao Li (Evan)
HighGo Software Co., Ltd.
https://www.highgo.com/
Attachments:
[application/octet-stream] v7-0001-ALTER-TABLE-emit-NOTICE-when-ONLY-is-omitted-but-.patch (39.5K, 2-v7-0001-ALTER-TABLE-emit-NOTICE-when-ONLY-is-omitted-but-.patch)
download | inline diff:
From e09d8d71015f379ede75dc9fbc280ff45137a6bd Mon Sep 17 00:00:00 2001
From: "Chao Li (Evan)" <[email protected]>
Date: Mon, 12 Jan 2026 16:56:58 +0800
Subject: [PATCH v7] ALTER TABLE: emit NOTICE when ONLY is omitted but no
partition recursion occurs
Some ALTER TABLE sub-commands do not recurse to partitions even when ONLY
is not specified on a partitioned table. This can be surprising, since
the default expectation is that commands propagate to all partitions.
Emit a NOTICE in such cases to make the behavior explicit, and advise how
to suppress the message or modify partitions individually.
Affected sub-commands include:
- ALTER COLUMN SET/RESET attribute_option
- ALTER COLUMN SET COMPRESSION
- ENABLE/DISABLE RULE
- ENABLE/DISABLE/FORCE/NO FORCE ROW LEVEL SECURITY
- REPLICA IDENTITY
- OWNER TO
- SET TABLESPACE
- SET SCHEMA
RENAME is intentionally excluded. Using ONLY (or not) has no effect for
RENAME, since relation names are independent by nature and there is no
expectation of recursion.
OF / NOT OF are also excluded. Using ONLY has no effect for these
commands, as they apply only to the partitioned table itself and not to
its partitions.
Regression tests are updated to use ONLY where appropriate.
Author: Chao Li <[email protected]>
Reviewed-by: David G. Johnston <[email protected]>
Reviewed-by: Greg Sabino Mullane <[email protected]>
Reviewed-by: Jim Jones <[email protected]>
Discussion: https://postgr.es/m/CAEoWx2=SLga-xH09Cq_PAvsHhQHrBK+V0vF821JKgzS=Bm0haA@mail.gmail.com
---
src/backend/commands/tablecmds.c | 138 ++++++++++++++++--
src/include/nodes/parsenodes.h | 1 +
src/test/regress/expected/alter_table.out | 84 ++++++++++-
src/test/regress/expected/cluster.out | 2 +-
src/test/regress/expected/merge.out | 4 +-
src/test/regress/expected/partition_merge.out | 2 +-
src/test/regress/expected/partition_split.out | 2 +-
src/test/regress/expected/privileges.out | 2 +-
src/test/regress/expected/rowsecurity.out | 4 +-
src/test/regress/expected/tablespace.out | 4 +-
src/test/regress/expected/update.out | 2 +-
src/test/regress/expected/vacuum.out | 6 +-
src/test/regress/sql/alter_table.sql | 64 +++++++-
src/test/regress/sql/cluster.sql | 2 +-
src/test/regress/sql/merge.sql | 4 +-
src/test/regress/sql/partition_merge.sql | 2 +-
src/test/regress/sql/partition_split.sql | 2 +-
src/test/regress/sql/privileges.sql | 2 +-
src/test/regress/sql/rowsecurity.sql | 4 +-
src/test/regress/sql/tablespace.sql | 4 +-
src/test/regress/sql/update.sql | 2 +-
src/test/regress/sql/vacuum.sql | 6 +-
22 files changed, 300 insertions(+), 43 deletions(-)
diff --git a/src/backend/commands/tablecmds.c b/src/backend/commands/tablecmds.c
index 85242dcc245..1e8ebde43e1 100644
--- a/src/backend/commands/tablecmds.c
+++ b/src/backend/commands/tablecmds.c
@@ -461,7 +461,8 @@ static void ATController(AlterTableStmt *parsetree,
AlterTableUtilityContext *context);
static void ATPrepCmd(List **wqueue, Relation rel, AlterTableCmd *cmd,
bool recurse, bool recursing, LOCKMODE lockmode,
- AlterTableUtilityContext *context);
+ AlterTableUtilityContext *context,
+ List **pendingNotice);
static void ATRewriteCatalogs(List **wqueue, LOCKMODE lockmode,
AlterTableUtilityContext *context);
static void ATExecCmd(List **wqueue, AlteredTableInfo *tab,
@@ -745,6 +746,10 @@ static void ATExecMergePartitions(List **wqueue, AlteredTableInfo *tab, Relation
static void ATExecSplitPartition(List **wqueue, AlteredTableInfo *tab,
Relation rel, PartitionCmd *cmd,
AlterTableUtilityContext *context);
+static void CollectPartitionNoRecurseNotice(AlterTableType cmdtype, Relation rel,
+ bool recurse, bool recursing,
+ List **pendingNotice);
+static void EmitPartitionNoRecurseNotice(List *pendingNotice);
/* ----------------------------------------------------------------
* DefineRelation
@@ -4886,13 +4891,14 @@ ATController(AlterTableStmt *parsetree,
{
List *wqueue = NIL;
ListCell *lcmd;
+ List *pendingNotice = NIL;
/* Phase 1: preliminary examination of commands, create work queue */
foreach(lcmd, cmds)
{
AlterTableCmd *cmd = (AlterTableCmd *) lfirst(lcmd);
- ATPrepCmd(&wqueue, rel, cmd, recurse, false, lockmode, context);
+ ATPrepCmd(&wqueue, rel, cmd, recurse, false, lockmode, context, &pendingNotice);
}
/* Close the relation, but keep lock until commit */
@@ -4903,6 +4909,9 @@ ATController(AlterTableStmt *parsetree,
/* Phase 3: scan/rewrite tables as needed, and run afterStmts */
ATRewriteTables(parsetree, &wqueue, lockmode, context);
+
+ /* Emit post-notice for partitions that were not recursed into. */
+ EmitPartitionNoRecurseNotice(pendingNotice);
}
/*
@@ -4917,7 +4926,8 @@ ATController(AlterTableStmt *parsetree,
static void
ATPrepCmd(List **wqueue, Relation rel, AlterTableCmd *cmd,
bool recurse, bool recursing, LOCKMODE lockmode,
- AlterTableUtilityContext *context)
+ AlterTableUtilityContext *context,
+ List **pendingNotice)
{
AlteredTableInfo *tab;
AlterTablePass pass = AT_PASS_UNSET;
@@ -5066,6 +5076,8 @@ ATPrepCmd(List **wqueue, Relation rel, AlterTableCmd *cmd,
ATT_MATVIEW | ATT_FOREIGN_TABLE);
/* This command never recurses */
pass = AT_PASS_MISC;
+ /* Emit a notice if needed */
+ CollectPartitionNoRecurseNotice(cmd->subtype, rel, recurse, recursing, pendingNotice);
break;
case AT_SetStorage: /* ALTER COLUMN SET STORAGE */
ATSimplePermissions(cmd->subtype, rel,
@@ -5081,6 +5093,8 @@ ATPrepCmd(List **wqueue, Relation rel, AlterTableCmd *cmd,
/* This command never recurses */
/* No command-specific prep needed */
pass = AT_PASS_MISC;
+ /* Emit a notice if needed */
+ CollectPartitionNoRecurseNotice(cmd->subtype, rel, recurse, recursing, pendingNotice);
break;
case AT_DropColumn: /* DROP COLUMN */
ATSimplePermissions(cmd->subtype, rel,
@@ -5148,6 +5162,8 @@ ATPrepCmd(List **wqueue, Relation rel, AlterTableCmd *cmd,
/* This command never recurses */
/* No command-specific prep needed */
pass = AT_PASS_MISC;
+ /* Emit a notice if needed */
+ CollectPartitionNoRecurseNotice(cmd->subtype, rel, recurse, recursing, pendingNotice);
break;
case AT_ClusterOn: /* CLUSTER ON */
case AT_DropCluster: /* SET WITHOUT CLUSTER */
@@ -5191,6 +5207,8 @@ ATPrepCmd(List **wqueue, Relation rel, AlterTableCmd *cmd,
/* This command never recurses */
ATPrepSetTableSpace(tab, rel, cmd->name, lockmode);
pass = AT_PASS_MISC; /* doesn't actually matter */
+ /* Emit a notice if needed */
+ CollectPartitionNoRecurseNotice(cmd->subtype, rel, recurse, recursing, pendingNotice);
break;
case AT_SetRelOptions: /* SET (...) */
case AT_ResetRelOptions: /* RESET (...) */
@@ -5200,6 +5218,7 @@ ATPrepCmd(List **wqueue, Relation rel, AlterTableCmd *cmd,
ATT_MATVIEW | ATT_INDEX);
/* This command never recurses */
/* No command-specific prep needed */
+ /* It will check for partitioned table at exec time */
pass = AT_PASS_MISC;
break;
case AT_AddInherit: /* INHERIT */
@@ -5237,8 +5256,8 @@ ATPrepCmd(List **wqueue, Relation rel, AlterTableCmd *cmd,
ATSimplePermissions(cmd->subtype, rel,
ATT_TABLE | ATT_PARTITIONED_TABLE | ATT_MATVIEW);
pass = AT_PASS_MISC;
- /* This command never recurses */
- /* No command-specific prep needed */
+ /* This command doesn't recurse to partitions, so notice if needed */
+ CollectPartitionNoRecurseNotice(cmd->subtype, rel, recurse, recursing, pendingNotice);
break;
case AT_EnableTrig: /* ENABLE TRIGGER variants */
case AT_EnableAlwaysTrig:
@@ -5259,17 +5278,30 @@ ATPrepCmd(List **wqueue, Relation rel, AlterTableCmd *cmd,
case AT_EnableAlwaysRule:
case AT_EnableReplicaRule:
case AT_DisableRule:
- case AT_AddOf: /* OF */
- case AT_DropOf: /* NOT OF */
- case AT_EnableRowSecurity:
+ case AT_EnableRowSecurity: /* ENABLE/DISABLE ROW SECURITY variants */
case AT_DisableRowSecurity:
- case AT_ForceRowSecurity:
+ case AT_ForceRowSecurity: /* FORCE/NO FORCE ROW SECURITY variants */
case AT_NoForceRowSecurity:
ATSimplePermissions(cmd->subtype, rel,
ATT_TABLE | ATT_PARTITIONED_TABLE);
/* These commands never recurse */
/* No command-specific prep needed */
pass = AT_PASS_MISC;
+ /* Emit a notice if needed */
+ CollectPartitionNoRecurseNotice(cmd->subtype, rel, recurse, recursing, pendingNotice);
+ break;
+ case AT_AddOf: /* OF */
+ case AT_DropOf: /* NOT OF */
+ ATSimplePermissions(cmd->subtype, rel,
+ ATT_TABLE | ATT_PARTITIONED_TABLE);
+ /* These commands never recurse */
+ /* No command-specific prep needed */
+
+ /*
+ * They only work on partitioned tables but child partitions, thus
+ * no need to emit a notice
+ */
+ pass = AT_PASS_MISC;
break;
case AT_GenericOptions:
ATSimplePermissions(cmd->subtype, rel, ATT_FOREIGN_TABLE);
@@ -6762,6 +6794,8 @@ alter_table_type_to_string(AlterTableType cmdtype)
return "ALTER COLUMN ... DROP IDENTITY";
case AT_ReAddStatistics:
return NULL; /* not real grammar */
+ case AT_SetSchema:
+ return "SET SCHEMA";
}
return NULL;
@@ -6883,7 +6917,7 @@ ATSimpleRecursion(List **wqueue, Relation rel,
/* find_all_inheritors already got lock */
childrel = relation_open(childrelid, NoLock);
CheckAlterTableIsSafe(childrel);
- ATPrepCmd(wqueue, childrel, cmd, false, true, lockmode, context);
+ ATPrepCmd(wqueue, childrel, cmd, false, true, lockmode, context, NULL);
relation_close(childrel, NoLock);
}
}
@@ -6946,7 +6980,7 @@ ATTypedTableRecursion(List **wqueue, Relation rel, AlterTableCmd *cmd,
childrel = relation_open(childrelid, lockmode);
CheckAlterTableIsSafe(childrel);
- ATPrepCmd(wqueue, childrel, cmd, true, true, lockmode, context);
+ ATPrepCmd(wqueue, childrel, cmd, true, true, lockmode, context, NULL);
relation_close(childrel, NoLock);
}
}
@@ -9592,7 +9626,7 @@ ATPrepAddPrimaryKey(List **wqueue, Relation rel, AlterTableCmd *cmd,
newcmd->recurse = true;
newcmd->def = (Node *) nnconstr;
- ATPrepCmd(wqueue, rel, newcmd, true, false, lockmode, context);
+ ATPrepCmd(wqueue, rel, newcmd, true, false, lockmode, context, NULL);
}
}
@@ -14673,7 +14707,7 @@ ATPrepAlterColumnType(List **wqueue,
errdetail("USING expression contains a whole-row table reference.")));
pfree(attmap);
}
- ATPrepCmd(wqueue, childrel, cmd, false, true, lockmode, context);
+ ATPrepCmd(wqueue, childrel, cmd, false, true, lockmode, context, NULL);
relation_close(childrel, NoLock);
}
}
@@ -18977,6 +19011,7 @@ AlterTableNamespace(AlterObjectSchemaStmt *stmt, Oid *oldschema)
RangeVar *newrv;
ObjectAddresses *objsMoved;
ObjectAddress myself;
+ List *pendingNotice = NIL;
relid = RangeVarGetRelidExtended(stmt->relation, AccessExclusiveLock,
stmt->missing_ok ? RVR_MISSING_OK : 0,
@@ -18993,6 +19028,13 @@ AlterTableNamespace(AlterObjectSchemaStmt *stmt, Oid *oldschema)
rel = relation_open(relid, NoLock);
+ /*
+ * SET SCHEMA doesn't recurse to children, emit a notice if ONLY is not
+ * specified. As this action doesn't go through ATPrepCmd, we have to emit
+ * the notice here.
+ */
+ CollectPartitionNoRecurseNotice(AT_SetSchema, rel, stmt->relation->inh, false, &pendingNotice);
+
oldNspOid = RelationGetNamespace(rel);
/* If it's an owned sequence, disallow moving it by itself. */
@@ -19030,6 +19072,8 @@ AlterTableNamespace(AlterObjectSchemaStmt *stmt, Oid *oldschema)
/* close rel, but keep lock until commit */
relation_close(rel, NoLock);
+ EmitPartitionNoRecurseNotice(pendingNotice);
+
return myself;
}
@@ -23381,3 +23425,71 @@ ATExecSplitPartition(List **wqueue, AlteredTableInfo *tab, Relation rel,
/* Restore the userid and security context. */
SetUserIdAndSecContext(save_userid, save_sec_context);
}
+
+/*
+ * When ONLY is not specified with a partitioned table, it is expected that the
+ * command recurses to all partitions. However, some sub-commands do not recurse.
+ * In such cases, emit a NOTICE to make this behavior explicit to the user.
+ */
+static void
+CollectPartitionNoRecurseNotice(AlterTableType cmdtype, Relation rel, bool recurse, bool recursing,
+ List **pendingNotice)
+{
+ if (pendingNotice == NULL)
+ return;
+
+ /* Only emit the notice at the top level of recursion */
+ if (rel->rd_rel->relkind == RELKIND_PARTITIONED_TABLE && recurse && !recursing)
+ {
+ PartitionDesc pd = RelationGetPartitionDesc(rel, true);
+ int nparts = pd->nparts;
+ const char *action_str;
+ char *notice_msg;
+ const ListCell *cell;
+ bool found = false;
+
+ /* Emit a notice only if there are partitions */
+ if (nparts == 0)
+ return;
+
+ action_str = alter_table_type_to_string(cmdtype);
+ notice_msg = psprintf(_("ALTER action %s on relation \"%s\" does not affect present partitions"),
+ action_str,
+ RelationGetRelationName(rel));
+
+ foreach(cell, *pendingNotice)
+ {
+ if (strcmp((char *) lfirst(cell), notice_msg) == 0)
+ {
+ pfree(notice_msg);
+ found = true;
+ break;
+ }
+ }
+ if (!found)
+ *pendingNotice = lappend(*pendingNotice, notice_msg);
+ }
+}
+
+static void
+EmitPartitionNoRecurseNotice(List *pendingNotice)
+{
+ ListCell *cell;
+ int len;
+ int i = 0;
+
+ len = list_length(pendingNotice);
+ foreach(cell, pendingNotice)
+ {
+ char *notice_msg = (char *) lfirst(cell);
+
+ /* Only emit the hint for the last notice */
+ i++;
+ ereport(NOTICE, errmsg("%s", notice_msg),
+ (i == len) ?
+ errhint("Partitions may be modified individually, or specify ONLY to suppress this message.")
+ : 0);
+ pfree(notice_msg);
+ }
+ list_free(pendingNotice);
+}
diff --git a/src/include/nodes/parsenodes.h b/src/include/nodes/parsenodes.h
index f3d32ef0188..1267fb59d3f 100644
--- a/src/include/nodes/parsenodes.h
+++ b/src/include/nodes/parsenodes.h
@@ -2511,6 +2511,7 @@ typedef enum AlterTableType
AT_SetIdentity, /* SET identity column options */
AT_DropIdentity, /* DROP IDENTITY */
AT_ReAddStatistics, /* internal to commands/tablecmds.c */
+ AT_SetSchema, /* SET SCHEMA */
} AlterTableType;
typedef struct AlterTableCmd /* one subcommand of an ALTER TABLE */
diff --git a/src/test/regress/expected/alter_table.out b/src/test/regress/expected/alter_table.out
index 5998c670aa3..bbb34bee938 100644
--- a/src/test/regress/expected/alter_table.out
+++ b/src/test/regress/expected/alter_table.out
@@ -4582,8 +4582,90 @@ SELECT * FROM list_parted;
---
(0 rows)
+CREATE TABLE list_parted4 (a int, b text) PARTITION BY LIST (a);
+CREATE TABLE list_parted4_1 PARTITION OF list_parted4 FOR VALUES IN (1);
+-- set column attribute on partitioned table without ONLY should get a notice
+ALTER TABLE list_parted4 ALTER COLUMN b SET (n_distinct = 0.2);
+NOTICE: ALTER action ALTER COLUMN ... SET on relation "list_parted4" does not affect present partitions
+HINT: Partitions may be modified individually, or specify ONLY to suppress this message.
+ALTER TABLE list_parted4 ALTER COLUMN b RESET (n_distinct);
+NOTICE: ALTER action ALTER COLUMN ... RESET on relation "list_parted4" does not affect present partitions
+HINT: Partitions may be modified individually, or specify ONLY to suppress this message.
+ALTER TABLE ONLY list_parted4 ALTER COLUMN b SET (n_distinct = 0.2);
+ALTER TABLE ONLY list_parted4 ALTER COLUMN b RESET (n_distinct);
+-- enable/disable rules on partitioned tables without ONLY should get a notice
+CREATE RULE list_parted4_rule AS ON INSERT TO list_parted4 DO INSTEAD NOTHING;
+ALTER TABLE list_parted4 DISABLE RULE list_parted4_rule;
+NOTICE: ALTER action DISABLE RULE on relation "list_parted4" does not affect present partitions
+HINT: Partitions may be modified individually, or specify ONLY to suppress this message.
+ALTER TABLE ONLY list_parted4 DISABLE RULE list_parted4_rule;
+ALTER TABLE list_parted4 ENABLE RULE list_parted4_rule;
+NOTICE: ALTER action ENABLE RULE on relation "list_parted4" does not affect present partitions
+HINT: Partitions may be modified individually, or specify ONLY to suppress this message.
+ALTER TABLE ONLY list_parted4 ENABLE RULE list_parted4_rule;
+DROP RULE list_parted4_rule ON list_parted4;
+-- enable/disable row level security on partitioned tables without ONLY should get a notice
+ALTER TABLE list_parted4 ENABLE ROW LEVEL SECURITY;
+NOTICE: ALTER action ENABLE ROW SECURITY on relation "list_parted4" does not affect present partitions
+HINT: Partitions may be modified individually, or specify ONLY to suppress this message.
+ALTER TABLE ONLY list_parted4 ENABLE ROW LEVEL SECURITY;
+ALTER TABLE list_parted4 DISABLE ROW LEVEL SECURITY;
+NOTICE: ALTER action DISABLE ROW SECURITY on relation "list_parted4" does not affect present partitions
+HINT: Partitions may be modified individually, or specify ONLY to suppress this message.
+ALTER TABLE ONLY list_parted4 DISABLE ROW LEVEL SECURITY;
+-- force/no force row level security on partitioned tables without ONLY should get a notice
+ALTER TABLE list_parted4 FORCE ROW LEVEL SECURITY;
+NOTICE: ALTER action FORCE ROW SECURITY on relation "list_parted4" does not affect present partitions
+HINT: Partitions may be modified individually, or specify ONLY to suppress this message.
+ALTER TABLE ONLY list_parted4 FORCE ROW LEVEL SECURITY;
+ALTER TABLE list_parted4 NO FORCE ROW LEVEL SECURITY;
+NOTICE: ALTER action NO FORCE ROW SECURITY on relation "list_parted4" does not affect present partitions
+HINT: Partitions may be modified individually, or specify ONLY to suppress this message.
+ALTER TABLE ONLY list_parted4 NO FORCE ROW LEVEL SECURITY;
+-- set replica identity on partitioned tables without ONLY should get a notice
+ALTER TABLE list_parted4 REPLICA IDENTITY FULL;
+NOTICE: ALTER action REPLICA IDENTITY on relation "list_parted4" does not affect present partitions
+HINT: Partitions may be modified individually, or specify ONLY to suppress this message.
+ALTER TABLE ONLY list_parted4 REPLICA IDENTITY FULL;
+ALTER TABLE list_parted4 REPLICA IDENTITY NOTHING;
+NOTICE: ALTER action REPLICA IDENTITY on relation "list_parted4" does not affect present partitions
+HINT: Partitions may be modified individually, or specify ONLY to suppress this message.
+ALTER TABLE ONLY list_parted4 REPLICA IDENTITY NOTHING;
+-- set compression on partitioned tables without ONLY should get a notice
+ALTER TABLE list_parted4 ALTER COLUMN b SET COMPRESSION pglz;
+NOTICE: ALTER action ALTER COLUMN ... SET COMPRESSION on relation "list_parted4" does not affect present partitions
+HINT: Partitions may be modified individually, or specify ONLY to suppress this message.
+ALTER TABLE ONLY list_parted4 ALTER COLUMN b SET COMPRESSION pglz;
+-- set owner on partitioned tables without ONLY should get a notice
+ALTER TABLE list_parted4 OWNER TO regress_alter_table_user1;
+NOTICE: ALTER action OWNER TO on relation "list_parted4" does not affect present partitions
+HINT: Partitions may be modified individually, or specify ONLY to suppress this message.
+ALTER TABLE ONLY list_parted4 OWNER TO regress_alter_table_user1;
+-- set schema on partitioned tables without ONLY should get a notice
+CREATE SCHEMA alter_table_test_schema;
+ALTER TABLE list_parted4 SET SCHEMA alter_table_test_schema;
+NOTICE: ALTER action SET SCHEMA on relation "list_parted4" does not affect present partitions
+HINT: Partitions may be modified individually, or specify ONLY to suppress this message.
+ALTER TABLE ONLY alter_table_test_schema.list_parted4 SET SCHEMA public;
+DROP SCHEMA alter_table_test_schema CASCADE;
+-- when there are multiple sub-command, notice should not duplicated
+ALTER TABLE list_parted4
+ ALTER COLUMN b SET COMPRESSION pglz,
+ ALTER COLUMN b SET COMPRESSION pglz, -- duplicate, but should not get duplicate notice
+ FORCE ROW LEVEL SECURITY,
+ REPLICA IDENTITY FULL,
+ OWNER TO regress_alter_table_user1;
+NOTICE: ALTER action ALTER COLUMN ... SET COMPRESSION on relation "list_parted4" does not affect present partitions
+NOTICE: ALTER action FORCE ROW SECURITY on relation "list_parted4" does not affect present partitions
+NOTICE: ALTER action REPLICA IDENTITY on relation "list_parted4" does not affect present partitions
+NOTICE: ALTER action OWNER TO on relation "list_parted4" does not affect present partitions
+HINT: Partitions may be modified individually, or specify ONLY to suppress this message.
+-- list_parted5 is a partitioned table that has no partition.
+CREATE TABLE list_parted5 (a int, b text) PARTITION BY LIST (a);
+-- as it has no partition, there should be no notice when altering it without ONLY
+ALTER TABLE list_parted5 FORCE ROW LEVEL SECURITY;
-- cleanup
-DROP TABLE list_parted, list_parted2, range_parted, list_parted3;
+DROP TABLE list_parted, list_parted2, range_parted, list_parted3, list_parted4, list_parted5;
DROP TABLE fail_def_part;
DROP TABLE hash_parted;
-- more tests for certain multi-level partitioning scenarios
diff --git a/src/test/regress/expected/cluster.out b/src/test/regress/expected/cluster.out
index 24b0b1a8fce..67d7ef6509d 100644
--- a/src/test/regress/expected/cluster.out
+++ b/src/test/regress/expected/cluster.out
@@ -547,7 +547,7 @@ SET SESSION AUTHORIZATION regress_ptnowner;
CLUSTER ptnowner USING ptnowner_i_idx;
ERROR: permission denied for table ptnowner
RESET SESSION AUTHORIZATION;
-ALTER TABLE ptnowner OWNER TO regress_ptnowner;
+ALTER TABLE ONLY ptnowner OWNER TO regress_ptnowner;
CREATE TEMP TABLE ptnowner_oldnodes AS
SELECT oid, relname, relfilenode FROM pg_partition_tree('ptnowner') AS tree
JOIN pg_class AS c ON c.oid=tree.relid;
diff --git a/src/test/regress/expected/merge.out b/src/test/regress/expected/merge.out
index 9cb1d87066a..bd6aac72336 100644
--- a/src/test/regress/expected/merge.out
+++ b/src/test/regress/expected/merge.out
@@ -2298,8 +2298,8 @@ SELECT * FROM pa_target ORDER BY tid, val;
ROLLBACK;
-- test RLS enforcement
BEGIN;
-ALTER TABLE pa_target ENABLE ROW LEVEL SECURITY;
-ALTER TABLE pa_target FORCE ROW LEVEL SECURITY;
+ALTER TABLE ONLY pa_target ENABLE ROW LEVEL SECURITY;
+ALTER TABLE ONLY pa_target FORCE ROW LEVEL SECURITY;
CREATE POLICY pa_target_pol ON pa_target USING (tid != 0);
MERGE INTO pa_target t
USING pa_source s
diff --git a/src/test/regress/expected/partition_merge.out b/src/test/regress/expected/partition_merge.out
index 925fe4f570a..f629fc7f3e5 100644
--- a/src/test/regress/expected/partition_merge.out
+++ b/src/test/regress/expected/partition_merge.out
@@ -810,7 +810,7 @@ SET SESSION AUTHORIZATION regress_partition_merge_bob;
ALTER TABLE t MERGE PARTITIONS (tp_0_1, tp_1_2) INTO tp_0_2;
ERROR: must be owner of table t
RESET SESSION AUTHORIZATION;
-ALTER TABLE t OWNER TO regress_partition_merge_bob;
+ALTER TABLE ONLY t OWNER TO regress_partition_merge_bob;
SET SESSION AUTHORIZATION regress_partition_merge_bob;
-- ERROR: must be owner of table tp_0_1
ALTER TABLE t MERGE PARTITIONS (tp_0_1, tp_1_2) INTO tp_0_2;
diff --git a/src/test/regress/expected/partition_split.out b/src/test/regress/expected/partition_split.out
index 4004efe0dac..421fa36ec0f 100644
--- a/src/test/regress/expected/partition_split.out
+++ b/src/test/regress/expected/partition_split.out
@@ -1372,7 +1372,7 @@ ALTER TABLE t SPLIT PARTITION tp_0_2 INTO
PARTITION tp_1_2 FOR VALUES FROM (1) TO (2)); --error
ERROR: must be owner of table t
RESET SESSION AUTHORIZATION;
-ALTER TABLE t OWNER TO regress_partition_split_bob;
+ALTER TABLE ONLY t OWNER TO regress_partition_split_bob;
SET SESSION AUTHORIZATION regress_partition_split_bob;
ALTER TABLE t SPLIT PARTITION tp_0_2 INTO
(PARTITION tp_0_1 FOR VALUES FROM (0) TO (1),
diff --git a/src/test/regress/expected/privileges.out b/src/test/regress/expected/privileges.out
index 7bc274566c3..877a1214963 100644
--- a/src/test/regress/expected/privileges.out
+++ b/src/test/regress/expected/privileges.out
@@ -1908,7 +1908,7 @@ SELECT brin_summarize_range('sro_brin', 0);
DROP TABLE sro_tab;
-- Check with a partitioned table
CREATE TABLE sro_ptab (a int) PARTITION BY RANGE (a);
-ALTER TABLE sro_ptab OWNER TO regress_sro_user;
+ALTER TABLE ONLY sro_ptab OWNER TO regress_sro_user;
CREATE TABLE sro_part PARTITION OF sro_ptab FOR VALUES FROM (1) TO (10);
ALTER TABLE sro_part OWNER TO regress_sro_user;
INSERT INTO sro_ptab VALUES (1), (2), (3);
diff --git a/src/test/regress/expected/rowsecurity.out b/src/test/regress/expected/rowsecurity.out
index 07d93e7def1..639c0065b0c 100644
--- a/src/test/regress/expected/rowsecurity.out
+++ b/src/test/regress/expected/rowsecurity.out
@@ -1233,7 +1233,7 @@ INSERT INTO part_document VALUES
( 8, 55, 2, 'regress_rls_carol', 'great satire'),
( 9, 11, 1, 'regress_rls_dave', 'awesome science fiction'),
(10, 99, 2, 'regress_rls_dave', 'awesome technology book');
-ALTER TABLE part_document ENABLE ROW LEVEL SECURITY;
+ALTER TABLE ONLY part_document ENABLE ROW LEVEL SECURITY;
-- Create policy on parent
-- user's security level must be higher than or equal to document's
CREATE POLICY pp1 ON part_document AS PERMISSIVE
@@ -4914,7 +4914,7 @@ CREATE TABLE rls_ptbl (a int) PARTITION BY RANGE (a);
CREATE TABLE rls_part PARTITION OF rls_ptbl FOR VALUES FROM (-100) TO (100);
INSERT INTO rls_ptbl SELECT x/10 FROM generate_series(1, 100) x;
ANALYZE rls_ptbl, rls_part;
-ALTER TABLE rls_ptbl ENABLE ROW LEVEL SECURITY;
+ALTER TABLE ONLY rls_ptbl ENABLE ROW LEVEL SECURITY;
ALTER TABLE rls_part ENABLE ROW LEVEL SECURITY;
GRANT SELECT ON rls_ptbl TO regress_rls_alice;
GRANT SELECT ON rls_part TO regress_rls_alice;
diff --git a/src/test/regress/expected/tablespace.out b/src/test/regress/expected/tablespace.out
index 185880a3217..bf437a92746 100644
--- a/src/test/regress/expected/tablespace.out
+++ b/src/test/regress/expected/tablespace.out
@@ -290,13 +290,13 @@ CREATE TABLE testschema.part_2 PARTITION OF testschema.part FOR VALUES IN (2);
SET default_tablespace TO pg_global;
CREATE TABLE testschema.part_3 PARTITION OF testschema.part FOR VALUES IN (3);
ERROR: only shared relations can be placed in pg_global tablespace
-ALTER TABLE testschema.part SET TABLESPACE regress_tblspace;
+ALTER TABLE ONLY testschema.part SET TABLESPACE regress_tblspace;
CREATE TABLE testschema.part_3 PARTITION OF testschema.part FOR VALUES IN (3);
CREATE TABLE testschema.part_4 PARTITION OF testschema.part FOR VALUES IN (4)
TABLESPACE pg_default;
CREATE TABLE testschema.part_56 PARTITION OF testschema.part FOR VALUES IN (5, 6)
PARTITION BY LIST (a);
-ALTER TABLE testschema.part SET TABLESPACE pg_default;
+ALTER TABLE ONLY testschema.part SET TABLESPACE pg_default;
CREATE TABLE testschema.part_78 PARTITION OF testschema.part FOR VALUES IN (7, 8)
PARTITION BY LIST (a);
ERROR: only shared relations can be placed in pg_global tablespace
diff --git a/src/test/regress/expected/update.out b/src/test/regress/expected/update.out
index eef2bac1cbf..584b40598eb 100644
--- a/src/test/regress/expected/update.out
+++ b/src/test/regress/expected/update.out
@@ -607,7 +607,7 @@ DROP TRIGGER trig_d15_20 ON part_d_15_20;
DROP FUNCTION func_parted_mod_b();
-- RLS policies with update-row-movement
-----------------------------------------
-ALTER TABLE range_parted ENABLE ROW LEVEL SECURITY;
+ALTER TABLE ONLY range_parted ENABLE ROW LEVEL SECURITY;
CREATE USER regress_range_parted_user;
GRANT ALL ON range_parted, mintab TO regress_range_parted_user;
CREATE POLICY seeall ON range_parted AS PERMISSIVE FOR SELECT USING (true);
diff --git a/src/test/regress/expected/vacuum.out b/src/test/regress/expected/vacuum.out
index d4696bc3325..f22bb2127e1 100644
--- a/src/test/regress/expected/vacuum.out
+++ b/src/test/regress/expected/vacuum.out
@@ -614,7 +614,7 @@ VACUUM (ANALYZE) vacowned_part2;
WARNING: permission denied to vacuum "vacowned_part2", skipping it
RESET ROLE;
-- Partitioned table and one partition owned by other user.
-ALTER TABLE vacowned_parted OWNER TO regress_vacuum;
+ALTER TABLE ONLY vacowned_parted OWNER TO regress_vacuum;
ALTER TABLE vacowned_part1 OWNER TO regress_vacuum;
SET ROLE regress_vacuum;
VACUUM vacowned_parted;
@@ -634,7 +634,7 @@ VACUUM (ANALYZE) vacowned_part2;
WARNING: permission denied to vacuum "vacowned_part2", skipping it
RESET ROLE;
-- Only one partition owned by other user.
-ALTER TABLE vacowned_parted OWNER TO CURRENT_USER;
+ALTER TABLE ONLY vacowned_parted OWNER TO CURRENT_USER;
SET ROLE regress_vacuum;
VACUUM vacowned_parted;
WARNING: permission denied to vacuum "vacowned_parted", skipping it
@@ -656,7 +656,7 @@ VACUUM (ANALYZE) vacowned_part2;
WARNING: permission denied to vacuum "vacowned_part2", skipping it
RESET ROLE;
-- Only partitioned table owned by other user.
-ALTER TABLE vacowned_parted OWNER TO regress_vacuum;
+ALTER TABLE ONLY vacowned_parted OWNER TO regress_vacuum;
ALTER TABLE vacowned_part1 OWNER TO CURRENT_USER;
SET ROLE regress_vacuum;
VACUUM vacowned_parted;
diff --git a/src/test/regress/sql/alter_table.sql b/src/test/regress/sql/alter_table.sql
index d6b6381ae5c..97f641c78da 100644
--- a/src/test/regress/sql/alter_table.sql
+++ b/src/test/regress/sql/alter_table.sql
@@ -2908,8 +2908,70 @@ ALTER TABLE list_parted2 ALTER COLUMN b TYPE text;
ALTER TABLE list_parted DROP COLUMN b;
SELECT * FROM list_parted;
+CREATE TABLE list_parted4 (a int, b text) PARTITION BY LIST (a);
+CREATE TABLE list_parted4_1 PARTITION OF list_parted4 FOR VALUES IN (1);
+
+-- set column attribute on partitioned table without ONLY should get a notice
+ALTER TABLE list_parted4 ALTER COLUMN b SET (n_distinct = 0.2);
+ALTER TABLE list_parted4 ALTER COLUMN b RESET (n_distinct);
+ALTER TABLE ONLY list_parted4 ALTER COLUMN b SET (n_distinct = 0.2);
+ALTER TABLE ONLY list_parted4 ALTER COLUMN b RESET (n_distinct);
+
+-- enable/disable rules on partitioned tables without ONLY should get a notice
+CREATE RULE list_parted4_rule AS ON INSERT TO list_parted4 DO INSTEAD NOTHING;
+ALTER TABLE list_parted4 DISABLE RULE list_parted4_rule;
+ALTER TABLE ONLY list_parted4 DISABLE RULE list_parted4_rule;
+ALTER TABLE list_parted4 ENABLE RULE list_parted4_rule;
+ALTER TABLE ONLY list_parted4 ENABLE RULE list_parted4_rule;
+DROP RULE list_parted4_rule ON list_parted4;
+
+-- enable/disable row level security on partitioned tables without ONLY should get a notice
+ALTER TABLE list_parted4 ENABLE ROW LEVEL SECURITY;
+ALTER TABLE ONLY list_parted4 ENABLE ROW LEVEL SECURITY;
+ALTER TABLE list_parted4 DISABLE ROW LEVEL SECURITY;
+ALTER TABLE ONLY list_parted4 DISABLE ROW LEVEL SECURITY;
+
+-- force/no force row level security on partitioned tables without ONLY should get a notice
+ALTER TABLE list_parted4 FORCE ROW LEVEL SECURITY;
+ALTER TABLE ONLY list_parted4 FORCE ROW LEVEL SECURITY;
+ALTER TABLE list_parted4 NO FORCE ROW LEVEL SECURITY;
+ALTER TABLE ONLY list_parted4 NO FORCE ROW LEVEL SECURITY;
+
+-- set replica identity on partitioned tables without ONLY should get a notice
+ALTER TABLE list_parted4 REPLICA IDENTITY FULL;
+ALTER TABLE ONLY list_parted4 REPLICA IDENTITY FULL;
+ALTER TABLE list_parted4 REPLICA IDENTITY NOTHING;
+ALTER TABLE ONLY list_parted4 REPLICA IDENTITY NOTHING;
+
+-- set compression on partitioned tables without ONLY should get a notice
+ALTER TABLE list_parted4 ALTER COLUMN b SET COMPRESSION pglz;
+ALTER TABLE ONLY list_parted4 ALTER COLUMN b SET COMPRESSION pglz;
+
+-- set owner on partitioned tables without ONLY should get a notice
+ALTER TABLE list_parted4 OWNER TO regress_alter_table_user1;
+ALTER TABLE ONLY list_parted4 OWNER TO regress_alter_table_user1;
+
+-- set schema on partitioned tables without ONLY should get a notice
+CREATE SCHEMA alter_table_test_schema;
+ALTER TABLE list_parted4 SET SCHEMA alter_table_test_schema;
+ALTER TABLE ONLY alter_table_test_schema.list_parted4 SET SCHEMA public;
+DROP SCHEMA alter_table_test_schema CASCADE;
+
+-- when there are multiple sub-command, notice should not duplicated
+ALTER TABLE list_parted4
+ ALTER COLUMN b SET COMPRESSION pglz,
+ ALTER COLUMN b SET COMPRESSION pglz, -- duplicate, but should not get duplicate notice
+ FORCE ROW LEVEL SECURITY,
+ REPLICA IDENTITY FULL,
+ OWNER TO regress_alter_table_user1;
+
+-- list_parted5 is a partitioned table that has no partition.
+CREATE TABLE list_parted5 (a int, b text) PARTITION BY LIST (a);
+-- as it has no partition, there should be no notice when altering it without ONLY
+ALTER TABLE list_parted5 FORCE ROW LEVEL SECURITY;
+
-- cleanup
-DROP TABLE list_parted, list_parted2, range_parted, list_parted3;
+DROP TABLE list_parted, list_parted2, range_parted, list_parted3, list_parted4, list_parted5;
DROP TABLE fail_def_part;
DROP TABLE hash_parted;
diff --git a/src/test/regress/sql/cluster.sql b/src/test/regress/sql/cluster.sql
index f90c6ec200b..3405e93ab0c 100644
--- a/src/test/regress/sql/cluster.sql
+++ b/src/test/regress/sql/cluster.sql
@@ -260,7 +260,7 @@ ALTER TABLE ptnowner1 OWNER TO regress_ptnowner;
SET SESSION AUTHORIZATION regress_ptnowner;
CLUSTER ptnowner USING ptnowner_i_idx;
RESET SESSION AUTHORIZATION;
-ALTER TABLE ptnowner OWNER TO regress_ptnowner;
+ALTER TABLE ONLY ptnowner OWNER TO regress_ptnowner;
CREATE TEMP TABLE ptnowner_oldnodes AS
SELECT oid, relname, relfilenode FROM pg_partition_tree('ptnowner') AS tree
JOIN pg_class AS c ON c.oid=tree.relid;
diff --git a/src/test/regress/sql/merge.sql b/src/test/regress/sql/merge.sql
index 2660b19f238..2bffcf7aa98 100644
--- a/src/test/regress/sql/merge.sql
+++ b/src/test/regress/sql/merge.sql
@@ -1422,8 +1422,8 @@ ROLLBACK;
-- test RLS enforcement
BEGIN;
-ALTER TABLE pa_target ENABLE ROW LEVEL SECURITY;
-ALTER TABLE pa_target FORCE ROW LEVEL SECURITY;
+ALTER TABLE ONLY pa_target ENABLE ROW LEVEL SECURITY;
+ALTER TABLE ONLY pa_target FORCE ROW LEVEL SECURITY;
CREATE POLICY pa_target_pol ON pa_target USING (tid != 0);
MERGE INTO pa_target t
USING pa_source s
diff --git a/src/test/regress/sql/partition_merge.sql b/src/test/regress/sql/partition_merge.sql
index a211fee2ad1..73bffb530dd 100644
--- a/src/test/regress/sql/partition_merge.sql
+++ b/src/test/regress/sql/partition_merge.sql
@@ -571,7 +571,7 @@ SET SESSION AUTHORIZATION regress_partition_merge_bob;
ALTER TABLE t MERGE PARTITIONS (tp_0_1, tp_1_2) INTO tp_0_2;
RESET SESSION AUTHORIZATION;
-ALTER TABLE t OWNER TO regress_partition_merge_bob;
+ALTER TABLE ONLY t OWNER TO regress_partition_merge_bob;
SET SESSION AUTHORIZATION regress_partition_merge_bob;
-- ERROR: must be owner of table tp_0_1
ALTER TABLE t MERGE PARTITIONS (tp_0_1, tp_1_2) INTO tp_0_2;
diff --git a/src/test/regress/sql/partition_split.sql b/src/test/regress/sql/partition_split.sql
index 37c6d730840..f4f9c7886e4 100644
--- a/src/test/regress/sql/partition_split.sql
+++ b/src/test/regress/sql/partition_split.sql
@@ -960,7 +960,7 @@ ALTER TABLE t SPLIT PARTITION tp_0_2 INTO
PARTITION tp_1_2 FOR VALUES FROM (1) TO (2)); --error
RESET SESSION AUTHORIZATION;
-ALTER TABLE t OWNER TO regress_partition_split_bob;
+ALTER TABLE ONLY t OWNER TO regress_partition_split_bob;
SET SESSION AUTHORIZATION regress_partition_split_bob;
ALTER TABLE t SPLIT PARTITION tp_0_2 INTO
(PARTITION tp_0_1 FOR VALUES FROM (0) TO (1),
diff --git a/src/test/regress/sql/privileges.sql b/src/test/regress/sql/privileges.sql
index 540f73ea9b1..eb73629a81f 100644
--- a/src/test/regress/sql/privileges.sql
+++ b/src/test/regress/sql/privileges.sql
@@ -1230,7 +1230,7 @@ SELECT brin_summarize_range('sro_brin', 0);
DROP TABLE sro_tab;
-- Check with a partitioned table
CREATE TABLE sro_ptab (a int) PARTITION BY RANGE (a);
-ALTER TABLE sro_ptab OWNER TO regress_sro_user;
+ALTER TABLE ONLY sro_ptab OWNER TO regress_sro_user;
CREATE TABLE sro_part PARTITION OF sro_ptab FOR VALUES FROM (1) TO (10);
ALTER TABLE sro_part OWNER TO regress_sro_user;
INSERT INTO sro_ptab VALUES (1), (2), (3);
diff --git a/src/test/regress/sql/rowsecurity.sql b/src/test/regress/sql/rowsecurity.sql
index 6b3566271df..1a2e80dade0 100644
--- a/src/test/regress/sql/rowsecurity.sql
+++ b/src/test/regress/sql/rowsecurity.sql
@@ -499,7 +499,7 @@ INSERT INTO part_document VALUES
( 9, 11, 1, 'regress_rls_dave', 'awesome science fiction'),
(10, 99, 2, 'regress_rls_dave', 'awesome technology book');
-ALTER TABLE part_document ENABLE ROW LEVEL SECURITY;
+ALTER TABLE ONLY part_document ENABLE ROW LEVEL SECURITY;
-- Create policy on parent
-- user's security level must be higher than or equal to document's
@@ -2414,7 +2414,7 @@ CREATE TABLE rls_part PARTITION OF rls_ptbl FOR VALUES FROM (-100) TO (100);
INSERT INTO rls_ptbl SELECT x/10 FROM generate_series(1, 100) x;
ANALYZE rls_ptbl, rls_part;
-ALTER TABLE rls_ptbl ENABLE ROW LEVEL SECURITY;
+ALTER TABLE ONLY rls_ptbl ENABLE ROW LEVEL SECURITY;
ALTER TABLE rls_part ENABLE ROW LEVEL SECURITY;
GRANT SELECT ON rls_ptbl TO regress_rls_alice;
GRANT SELECT ON rls_part TO regress_rls_alice;
diff --git a/src/test/regress/sql/tablespace.sql b/src/test/regress/sql/tablespace.sql
index c43a59e5957..8eff2914a5b 100644
--- a/src/test/regress/sql/tablespace.sql
+++ b/src/test/regress/sql/tablespace.sql
@@ -195,13 +195,13 @@ SET default_tablespace TO regress_tblspace;
CREATE TABLE testschema.part_2 PARTITION OF testschema.part FOR VALUES IN (2);
SET default_tablespace TO pg_global;
CREATE TABLE testschema.part_3 PARTITION OF testschema.part FOR VALUES IN (3);
-ALTER TABLE testschema.part SET TABLESPACE regress_tblspace;
+ALTER TABLE ONLY testschema.part SET TABLESPACE regress_tblspace;
CREATE TABLE testschema.part_3 PARTITION OF testschema.part FOR VALUES IN (3);
CREATE TABLE testschema.part_4 PARTITION OF testschema.part FOR VALUES IN (4)
TABLESPACE pg_default;
CREATE TABLE testschema.part_56 PARTITION OF testschema.part FOR VALUES IN (5, 6)
PARTITION BY LIST (a);
-ALTER TABLE testschema.part SET TABLESPACE pg_default;
+ALTER TABLE ONLY testschema.part SET TABLESPACE pg_default;
CREATE TABLE testschema.part_78 PARTITION OF testschema.part FOR VALUES IN (7, 8)
PARTITION BY LIST (a);
CREATE TABLE testschema.part_910 PARTITION OF testschema.part FOR VALUES IN (9, 10)
diff --git a/src/test/regress/sql/update.sql b/src/test/regress/sql/update.sql
index 8b4707eb9c3..56b1edb00ce 100644
--- a/src/test/regress/sql/update.sql
+++ b/src/test/regress/sql/update.sql
@@ -341,7 +341,7 @@ DROP FUNCTION func_parted_mod_b();
-- RLS policies with update-row-movement
-----------------------------------------
-ALTER TABLE range_parted ENABLE ROW LEVEL SECURITY;
+ALTER TABLE ONLY range_parted ENABLE ROW LEVEL SECURITY;
CREATE USER regress_range_parted_user;
GRANT ALL ON range_parted, mintab TO regress_range_parted_user;
CREATE POLICY seeall ON range_parted AS PERMISSIVE FOR SELECT USING (true);
diff --git a/src/test/regress/sql/vacuum.sql b/src/test/regress/sql/vacuum.sql
index 247b8e23b23..acff5824ebb 100644
--- a/src/test/regress/sql/vacuum.sql
+++ b/src/test/regress/sql/vacuum.sql
@@ -452,7 +452,7 @@ VACUUM (ANALYZE) vacowned_part1;
VACUUM (ANALYZE) vacowned_part2;
RESET ROLE;
-- Partitioned table and one partition owned by other user.
-ALTER TABLE vacowned_parted OWNER TO regress_vacuum;
+ALTER TABLE ONLY vacowned_parted OWNER TO regress_vacuum;
ALTER TABLE vacowned_part1 OWNER TO regress_vacuum;
SET ROLE regress_vacuum;
VACUUM vacowned_parted;
@@ -466,7 +466,7 @@ VACUUM (ANALYZE) vacowned_part1;
VACUUM (ANALYZE) vacowned_part2;
RESET ROLE;
-- Only one partition owned by other user.
-ALTER TABLE vacowned_parted OWNER TO CURRENT_USER;
+ALTER TABLE ONLY vacowned_parted OWNER TO CURRENT_USER;
SET ROLE regress_vacuum;
VACUUM vacowned_parted;
VACUUM vacowned_part1;
@@ -479,7 +479,7 @@ VACUUM (ANALYZE) vacowned_part1;
VACUUM (ANALYZE) vacowned_part2;
RESET ROLE;
-- Only partitioned table owned by other user.
-ALTER TABLE vacowned_parted OWNER TO regress_vacuum;
+ALTER TABLE ONLY vacowned_parted OWNER TO regress_vacuum;
ALTER TABLE vacowned_part1 OWNER TO CURRENT_USER;
SET ROLE regress_vacuum;
VACUUM vacowned_parted;
--
2.50.1 (Apple Git-155)
view thread (19+ messages) latest in thread
reply
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Reply to all the recipients using the --to and --cc options:
reply via email
To: [email protected]
Cc: [email protected], [email protected], [email protected], [email protected], [email protected]
Subject: Re: ALTER TABLE: warn when actions do not recurse to partitions
In-Reply-To: <[email protected]>
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
This inbox is served by agora; see mirroring instructions
for how to clone and mirror all data and code used for this inbox