review comments

yugabyte · Feb 5, 2025 · e10375e · e10375e
1 parent 28eee46
commit e10375e
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 12 deletions.
diff --git a/yb-voyager/src/query/queryissue/issues_ddl.go b/yb-voyager/src/query/queryissue/issues_ddl.go
@@ -634,14 +634,15 @@ func NewUniqueNullsNotDistinctIssue(objectType string, objectName string, sqlSta
 }
 
 /*
-Database options works on 2.25 but not marking it as supported in 2.25 for now but as per this ticket 
- https://github.com/yugabyte/yugabyte-db/issues/25541, DB will be blocking this support so its not supported technically
+Database options works on 2.25 but not marking it as supported in 2.25 for now but as per this ticket
+
+	https://github.com/yugabyte/yugabyte-db/issues/25541, DB will be blocking this support so its not supported technically
 */
 var databaseOptionsPG15Issue = issue.Issue{
 	Type:        DATABASE_OPTIONS_PG15,
 	Name:        "Database options",
 	Impact:      constants.IMPACT_LEVEL_2,
-	Description: "Database options (%s) introduced in PostgreSQL 12 and later are not supported yet in YugabyteDB.",
+	Description: DATABASE_OPTIONS_DESCRIPTION,
 	Suggestion:  "",
 	GH:          "https://github.com/yugabyte/yugabyte-db/issues/25575",
 	DocsLink:    "https://docs.yugabyte.com/preview/yugabyte-voyager/known-issues/postgresql/#postgresql-12-and-later-features",
@@ -660,7 +661,7 @@ var databaseOptionsPG17Issue = issue.Issue{
 	Type:        DATABASE_OPTIONS_PG17,
 	Name:        "Database options",
 	Impact:      constants.IMPACT_LEVEL_2,
-	Description: "Database options (%s) introduced in PostgreSQL 12 and later are not supported yet in YugabyteDB.",
+	Description: DATABASE_OPTIONS_DESCRIPTION,
 	Suggestion:  "",
 	GH:          "https://github.com/yugabyte/yugabyte-db/issues/25575",
 	DocsLink:    "https://docs.yugabyte.com/preview/yugabyte-voyager/known-issues/postgresql/#postgresql-12-and-later-features",

diff --git a/yb-voyager/src/query/queryparser/helpers_protomsg.go b/yb-voyager/src/query/queryparser/helpers_protomsg.go
@@ -415,14 +415,6 @@ func ProtoAsAConstNode(msg protoreflect.Message) (*pg_query.A_Const, error) {
 	return aConstNode, nil
 }
 
-func ProtoAsCreateDBStmtNode(msg protoreflect.Message) (*pg_query.CreatedbStmt, error) {
-	node, ok := msg.Interface().(*pg_query.CreatedbStmt)
-	if !ok {
-		return nil, fmt.Errorf("failed to cast msg to %s", PG_QUERY_CREATEDB_STMT_NODE)
-	}
-	return node, nil
-}
-
 func ProtoAsCTENode(msg protoreflect.Message) (*pg_query.CommonTableExpr, error) {
 	cteNode, ok := msg.Interface().(*pg_query.CommonTableExpr)
 	if !ok {