From 25a780af8188a47f1e1b9a755d7d263df79d785c Mon Sep 17 00:00:00 2001 From: aykutbozkurt Date: Mon, 1 Aug 2022 18:39:34 +0300 Subject: [PATCH] RoleSpecToObjectAddress should return a valid address for the role ALL --- src/backend/distributed/commands/role.c | 7 +++++++ 1 file changed, 7 insertions(+) diff --git a/src/backend/distributed/commands/role.c b/src/backend/distributed/commands/role.c index dcec5c2cc..a1c30d9f2 100644 --- a/src/backend/distributed/commands/role.c +++ b/src/backend/distributed/commands/role.c @@ -124,6 +124,13 @@ RoleSpecToObjectAddress(RoleSpec *role, bool missing_ok) Oid roleOid = get_rolespec_oid(role, missing_ok); ObjectAddressSet(*address, AuthIdRelationId, roleOid); } + else + { + /* + * If rolespec is null, role can be 'ALL'. We should return a pseudo-valid oid. + */ + ObjectAddressSet(*address, AuthIdRelationId, OID_MAX); + } return list_make1(address); }