From ef582cef3b953e02a5d53011617efb8c16cc07bb Mon Sep 17 00:00:00 2001 From: Ruslan Talpa Date: Thu, 24 Jul 2025 11:17:18 +0300 Subject: [PATCH] fix errors --- pgxn/neon/neon_ddl_handler.c | 28 ++++++++++++++++++---------- 1 file changed, 18 insertions(+), 10 deletions(-) diff --git a/pgxn/neon/neon_ddl_handler.c b/pgxn/neon/neon_ddl_handler.c index 245f7bad68..7c18c016bb 100644 --- a/pgxn/neon/neon_ddl_handler.c +++ b/pgxn/neon/neon_ddl_handler.c @@ -573,6 +573,13 @@ IsPrivilegedRole(const char *role_name) return strcmp(role_name, privileged_role_name) == 0; } +static void +HandleOtherDDLCommand() +{ + InitCurrentDdlTableIfNeeded(); + CurrentDdlTable->other_ddl_count++; +} + static void HandleCreateDb(CreatedbStmt *stmt) { @@ -622,8 +629,11 @@ HandleAlterOwner(AlterOwnerStmt *stmt) DbEntry *entry; const char *new_owner; - if (stmt->objectType != OBJECT_DATABASE) - return; + if (stmt->objectType != OBJECT_DATABASE){ + HandleOtherDDLCommand(); + return ; + } + InitDbTableIfNeeded(); name = strVal(stmt->object); @@ -850,14 +860,12 @@ HandleRename(RenameStmt *stmt) return HandleDbRename(stmt); else if (stmt->renameType == OBJECT_ROLE) return HandleRoleRename(stmt); + else { + HandleOtherDDLCommand(); + return; + } } -static void -HandleOtherDDLCommand() -{ - InitCurrentDdlTableIfNeeded(); - CurrentDdlTable->other_ddl_count++; -} /* * Support for Event Triggers. @@ -1386,8 +1394,8 @@ NeonProcessUtility( // Generic Operations (object type dependent) case T_DropStmt: // DROP (tables, views, functions, etc.) - case T_RenameStmt: // ALTER ... RENAME TO - case T_AlterOwnerStmt: // ALTER ... OWNER TO + // case T_RenameStmt: // ALTER ... RENAME TO + // case T_AlterOwnerStmt: // ALTER ... OWNER TO case T_AlterObjectDependsStmt: // ALTER ... DEPENDS ON case T_AlterObjectSchemaStmt: // ALTER ... SET SCHEMA case T_CommentStmt: // COMMENT ON