Revert "store children with database backend"#8475
Conversation
This reverts commit 51b2846.
Codecov ReportPatch coverage has no change and project coverage change:
Additional details and impacted files@@ Coverage Diff @@
## main #8475 +/- ##
==========================================
- Coverage 87.44% 87.43% -0.01%
==========================================
Files 148 148
Lines 18488 18487 -1
Branches 3155 3155
==========================================
- Hits 16166 16165 -1
Misses 2033 2033
Partials 289 289
Flags with carried forward coverage won't be shown. Click here to find out more.
☔ View full report in Codecov by Sentry. |
Nusnus
left a comment
There was a problem hiding this comment.
Agreed. This was a good change but unfortunately not complete. Once we have the remaining changes we can welcome this back.
|
@aaronst we're reverting the change in Ping me if you need any help/review and I'll prioritize it for me due to this situation. |
@aaronst Is it reasonable to ask you please for the remaining changes (the migration) so we can complete it instead of documenting the gap? |
|
@Nusnus I’m happy to help with the migration, but this bug still needs to be documented if the fix is being reverted. |
|
I will be happy to accept a
I'm still open to a doc fix, until the code is fixed |
Reverts #8338
@Nusnus we might revert this for now and reintroduce in future with a proper migration support. for migration support I can work