diff --git a/yb-voyager/cmd/analyzeSchema.go b/yb-voyager/cmd/analyzeSchema.go index 79efec3b8..55ea06e1c 100644 --- a/yb-voyager/cmd/analyzeSchema.go +++ b/yb-voyager/cmd/analyzeSchema.go @@ -678,7 +678,7 @@ func convertIssueInstanceToAnalyzeIssue(issueInstance queryissue.QueryIssue, fil ObjectName: displayObjectName, Reason: issueInstance.Name, Type: issueInstance.Type, - Impact: issueInstance.GetImpact(), + Impact: issueInstance.Impact, SqlStatement: issueInstance.SqlStatement, DocsLink: issueInstance.DocsLink, FilePath: fileName, diff --git a/yb-voyager/src/issue/issue.go b/yb-voyager/src/issue/issue.go index f1c9358c2..47bf663fb 100644 --- a/yb-voyager/src/issue/issue.go +++ b/yb-voyager/src/issue/issue.go @@ -17,8 +17,6 @@ limitations under the License. package issue import ( - "github.com/samber/lo" - "github.com/yugabyte/yb-voyager/yb-voyager/src/constants" "github.com/yugabyte/yb-voyager/yb-voyager/src/ybversion" ) @@ -44,11 +42,6 @@ func (i Issue) IsFixedIn(v *ybversion.YBVersion) (bool, error) { return v.GreaterThanOrEqual(minVersionFixedInSeries), nil } -func (i Issue) GetImpact() string { - // Default value as IMPACT_LEVEL_1 if not defined in issue - return lo.Ternary(i.Impact != "", i.Impact, constants.IMPACT_LEVEL_1) -} - /* Dynamic Impact Determination (TODO) - We can define the impact calculator function based on issue type wherever/whenever needed @@ -72,7 +65,6 @@ func (i Issue) GetImpact() string { return "LEVEL_3" } - // Update existing GetImpact() method func (i Issue) GetImpact(stats *PgStats) string { if calc, ok := impactCalculators[i.Type]; ok { return calc(i, stats)