Apply suggestions from code review

Co-authored-by: Onur Tirtir <onurcantirtir@gmail.com>

src/backend/distributed/commands/database.c

@@ -699,12 +699,8 @@ List *
 PreprocessAlterDatabaseOwnerStmt(Node *node, const char *queryString,
 								 ProcessUtilityContext processUtilityContext)
 {
-	const DistributeObjectOps *ops = GetDistributeObjectOps(node);
-	Assert(ops != NULL);
-
-	if (ops->featureFlag && *ops->featureFlag == false)
+	if (!EnableAlterDatabaseOwner)
 	{
-		/* not propagating when a configured feature flag is turned off by the user */
 		return NIL;
 	}
 
@@ -717,6 +713,7 @@ PreprocessAlterDatabaseOwnerStmt(Node *node, const char *queryString,
 	{
 		return NIL;
 	}
+
 	EnsurePropagationToCoordinator();
 
 	AlterOwnerStmt *stmt = (AlterOwnerStmt *) node;

src/backend/distributed/commands/non_main_db_distribute_object_ops.c

@@ -76,6 +76,10 @@ static bool GrantStmtCheckSupportedObjectType(Node *node);
 static bool SecLabelStmtCheckSupportedObjectType(Node *node);
 static bool AlterDbRenameCheckSupportedObjectType(Node *node);
 static bool AlterDbOwnerCheckSupportedObjectType(Node *node);
+
+/*
+ * cannotBeExecutedInTransaction callbacks for OperationArray.
+ */
 static bool CannotBeExecutedInTransaction_True(Node *node);
 static bool CannotBeExecutedInTransaction_False(Node *node);
 static bool AlterDbCannotBeExecutedInTransaction(Node *node);

src/include/distributed/commands.h

@@ -236,7 +236,6 @@ extern List * PreprocessAlterDatabaseRefreshCollStmt(Node *node, const char *que
 													 processUtilityContext);
 extern List * PreprocessAlterDatabaseOwnerStmt(Node *node, const char *queryString,
 											   ProcessUtilityContext processUtilityContext);
-
 extern List * GetDatabaseMetadataSyncCommands(Oid dbOid);