Depends on D6956 to support all objects
Details
Details
tested in an empty Docker
Diff Detail
Diff Detail
- Repository
- rDSNIP Code snippets
- Lint
Automatic diff as part of commit; lint not applicable. - Unit
Automatic diff as part of commit; unit tests not applicable.
Event Timeline
vlorentz/recover_corrupt_objects.py | ||
---|---|---|
75 | I'm not sure that's worth bothering. If you want to do it, the takedown snippet has the proper code. You have to:
| |
79–80 | I think these statements need to be swapped to avoid a reject due to constraints (or add a cascade to the first statement) |
vlorentz/recover_corrupt_objects.py | ||
---|---|---|
79–80 | Ah, right. I tested it with the default docker db, looks like it doesn't have the flavor that does the check |
Comment Actions
- recover_corrupt_objects.py: Double-check insertion (this allowed me to find the bug fixed by D7024)
Comment Actions
I'm not 100% convinced we need to recheck the objects at every addition (within a transaction that can still fail to commit) instead of afterwards, but it doesn't /hurt/ either. We'll make a full pass on all objects again later anyway.