added impact for issues missed before

yugabyte · Jan 7, 2025 · 3cee8cf · 3cee8cf
1 parent 134b147
commit 3cee8cf
Show file tree

Hide file tree

Showing 3 changed files with 8 additions and 0 deletions.
diff --git a/yb-voyager/cmd/assessMigrationCommand.go b/yb-voyager/cmd/assessMigrationCommand.go
@@ -1175,6 +1175,7 @@ func fetchUnsupportedObjectTypes() ([]UnsupportedFeature, error) {
 			referenceOrTablePartitionPresent = true
 			unsupportedPartitionTypes = append(unsupportedPartitionTypes, ObjectInfo{ObjectName: fmt.Sprintf("Table Name: %s, Partition Method: %s", objectName, objectType)})
 
+			// For oracle migration complexity comes from ora2pg, so defining Impact not required right now
 			assessmentReport.AppendIssues(AssessmentIssue{
 				Category:   constants.FEATURE,
 				Type:       "", // TODO

diff --git a/yb-voyager/src/query/queryissue/issues_ddl.go b/yb-voyager/src/query/queryissue/issues_ddl.go
@@ -503,6 +503,7 @@ func NewSecurityInvokerViewIssue(objectType string, objectName string, SqlStatem
 var foreignKeyReferencesPartitionedTableIssue = issue.Issue{
 	Type:       FOREIGN_KEY_REFERENCES_PARTITIONED_TABLE,
 	Name:       FOREIGN_KEY_REFERENCES_PARTITIONED_TABLE_NAME,
+	Impact:     constants.IMPACT_LEVEL_1,
 	Suggestion: "No workaround available ",
 	GH:         "", // TODO
 	DocsLink:   "", // TODO

diff --git a/yb-voyager/src/query/queryissue/issues_dml.go b/yb-voyager/src/query/queryissue/issues_dml.go
@@ -82,6 +82,7 @@ func NewRegexFunctionsIssue(objectType string, objectName string, sqlStatement s
 var aggregateFunctionIssue = issue.Issue{
 	Type:            AGGREGATE_FUNCTION,
 	Name:        AGGREGATION_FUNCTIONS_NAME,
+	Impact:      constants.IMPACT_LEVEL_2,
 	Description: "any_value, range_agg and range_intersect_agg functions not supported yet in YugabyteDB",
 	Suggestion:      "",
 	GH:              "",
@@ -99,6 +100,7 @@ func NewAggregationFunctionIssue(objectType string, objectName string, sqlStatem
 var jsonConstructorFunctionsIssue = issue.Issue{
 	Type:        JSON_CONSTRUCTOR_FUNCTION,
 	Name:        JSON_CONSTRUCTOR_FUNCTION_NAME,
+	Impact:      constants.IMPACT_LEVEL_2,
 	Description: "Postgresql 17 features not supported yet in YugabyteDB",
 	Suggestion:  "",
 	GH:          "",
@@ -116,6 +118,7 @@ func NewJsonConstructorFunctionIssue(objectType string, objectName string, sqlSt
 var jsonQueryFunctionIssue = issue.Issue{
 	Type:        JSON_QUERY_FUNCTION,
 	Name:        JSON_QUERY_FUNCTIONS_NAME,
+	Impact:      constants.IMPACT_LEVEL_2,
 	Description: "Postgresql 17 features not supported yet in YugabyteDB",
 	Suggestion:  "",
 	GH:          "",
@@ -133,6 +136,7 @@ func NewJsonQueryFunctionIssue(objectType string, objectName string, sqlStatemen
 var loFunctionsIssue = issue.Issue{
 	Type:        LARGE_OBJECT_FUNCTIONS,
 	Name:        LARGE_OBJECT_FUNCTIONS_NAME,
+	Impact:      constants.IMPACT_LEVEL_2,
 	Description: "Large Objects functions are not supported in YugabyteDB",
 	Suggestion:  "Large objects functions are not yet supported in YugabyteDB, no workaround available right now",
 	GH:          "https://github.com/yugabyte/yugabyte-db/issues/25318",
@@ -150,6 +154,7 @@ func NewLOFuntionsIssue(objectType string, objectName string, sqlStatement strin
 var jsonPredicateIssue = issue.Issue{
 	Type:            JSON_TYPE_PREDICATE,
 	Name:        JSON_TYPE_PREDICATE_NAME,
+	Impact:      constants.IMPACT_LEVEL_2,
 	Description: "IS JSON predicate expressions not supported yet in YugabyteDB",
 	Suggestion:      "",
 	GH:              "",
@@ -191,6 +196,7 @@ func NewCopyOnErrorIssue(objectType string, objectName string, sqlStatement stri
 var fetchWithTiesIssue = issue.Issue{
 	Type:        FETCH_WITH_TIES,
 	Name:        "FETCH .. WITH TIES",
+	Impact:      constants.IMPACT_LEVEL_2,
 	Description: "FETCH .. WITH TIES is not supported in YugabyteDB",
 	Suggestion:  "No workaround available right now",
 	GH:          "",