On Thu, Apr 27, 2023 at 04:59:13PM +0800, Richard Guo wrote:
> In this case the CreateSchemaStmtContext.schemaname is NULL since it is
> not explicitly specified, while the schemaname in the schema element is
> not NULL as it is specified, and setSchemaName cannot copy with such
> situation. Maybe we should check against RoleSpec.rolename in this case
> since that is also the schema's name?
In this case, it is cleaner to just set the schema name in
CreateSchemaStmtContext.schemaname to the role in the RoleSpec if there
is no schema set in the query, because the schema name will have the
same name as the role. That also makes the handling of each element
in schemaElts simpler.
The regression tests cruelly lacks of checks here. This is not a
pattern of CREATE SCHEMA known a lot, but we should do better.
--
Michael
