SYN-7334: resolve view bug in ddefs and implement a storage migration

[invisig0th]

No description provided.

[codecov]

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 97.73%. Comparing base (d94ccac) to head (05557c4).
⚠️ Report is 5 commits behind head on master.

Additional details and impacted files

@@            Coverage Diff             @@
##           master    #4771      +/-   ##
==========================================
- Coverage   97.73%   97.73%   -0.01%     
==========================================
  Files         298      298              
  Lines       62363    62383      +20     
==========================================
+ Hits        60952    60968      +16     
- Misses       1411     1415       +4     

Flag	Coverage Δ
linux	97.65% <100.00%> (-0.03%)	⬇️
linux_replay	93.41% <100.00%> (+<0.01%)	⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

[MichaelSquires]

I think we should be validating the ddef before adding it.

I also noticed a few comments from @vEpiphyte in the ticket that don't seem to be addressed: https://vertexproject.atlassian.net/browse/SYN-7334?focusedCommentId=18032

[invisig0th]

I think we should be validating the ddef before adding it.

Added validation 👍

I also noticed a few comments from @vEpiphyte in the ticket that don't seem to be addressed: https://vertexproject.atlassian.net/browse/SYN-7334?focusedCommentId=18032

The comments represent work that seems way outside the scope for this ticket. We should probably create another ticket, i suspect it will be a larger LOE than this entire Story captured originally...