You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
badgerwithagun
changed the title
upgrade package and AssumedUUIDs.txt file look orphaned and should be removed
AssumedUUIDs.file in upgrade package resources looks suspicious
Jul 2, 2017
AssumedUUIDs.txt is a legacy of Alfa first introduced the UpgradeAudit table to track upgrade steps. It should at least be moved out of Morf, and possibly be removed entirely.
UpgradePathFinder contains a logic for ignoring UUIDs loaded from this file. I imagine this has been added for backwards compatibility.
Do we need to support a scenario when some UUIDs are ignored in the upgrade process ? If the answer is yes then we could think about making this part of the public API.
Another option is to make wrap it up in the unsupported API to be removed in the future once obsolete UUID are no longer a problem.
I guess there is some historical reason why this exists. I have my doubts as to whether anyone other than us would need it going forward, so whatever we implement here will probably never be a supported API. ideally we'd factor it out entirely. @tsg21 - thoughts?
https://github.com/alfasoftware/morf/blob/master/morf-core/src/main/resources/org/alfasoftware/morf/upgrade/AssumedUUIDs.txt
The text was updated successfully, but these errors were encountered: