Github user sandhyasun commented on a diff in the pull request:
https://github.com/apache/trafodion/pull/1721#discussion_r225719386
--- Diff: core/sql/sqlcomp/CmpSeabaseDDLupgrade.cpp ---
@@ -1657,7 +1717,7 @@ short
CmpSeabaseMDupgrade::executeSeabaseMDupgrade(CmpDDLwithStatusInfo *mdui,
{
*CmpCommon::diags() << DgSqlCode(-20123);
- mdui->setStep(UPGRADE_FAILED_RESTORE_OLD_REPOS);
+ mdui->setStep(UPGRADE_FAILED_RESTORE_OLD_LIBRARIES);
--- End diff --
We ar ein the phase of rewinding the upgrade. Unless the "needsUpgrade"
flag is set, the restore steps wont do anything. So we are ok. We just need to
start with last step and rewind but some of the in between steps may not do any
rewinding.
---
