Remove ChangeLoggedModel from StagedChange #8916

Closed
opened 2025-12-29 20:42:49 +01:00 by adam · 0 comments
Owner

Originally created by @arthanson on GitHub (Dec 4, 2023).

Originally assigned to: @arthanson on GitHub.

Proposed Changes

Remove ChangeLoggedModel mixin from the StagedChange model. Marking this as a deprecation as it would remove functionality.

Justification

As per #13640 When doing staged change API it can result in numerous ChangeLoggedModels being produced - For example when ingesting some customer information using the staging method there can be up to 100k+ objects being altered. Deleting these takes very long due to the ChangeLoggedModel process.

Impact

Would regress in that no more ChangeLoggedModels would be produced for StagedChanges.

Originally created by @arthanson on GitHub (Dec 4, 2023). Originally assigned to: @arthanson on GitHub. ### Proposed Changes Remove ChangeLoggedModel mixin from the StagedChange model. Marking this as a deprecation as it would remove functionality. ### Justification As per #13640 When doing staged change API it can result in numerous ChangeLoggedModels being produced - For example when ingesting some customer information using the staging method there can be up to 100k+ objects being altered. Deleting these takes very long due to the ChangeLoggedModel process. ### Impact Would regress in that no more ChangeLoggedModels would be produced for StagedChanges.
adam added the status: acceptedtype: deprecation labels 2025-12-29 20:42:50 +01:00
adam closed this issue 2025-12-29 20:42:50 +01:00
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: starred/netbox#8916