Project

Profile

Help

Issue #4306

The BulkCreateManager.bulk_get_or_create() breaks ourter transactions.

Added by jortel@redhat.com 9 months ago. Updated 6 months ago.

Status:
MODIFIED
Priority:
Normal
Category:
-
Sprint/Milestone:
Start date:
Due date:
Severity:
2. Medium
Version:
Platform Release:
Blocks Release:
OS:
Backwards Incompatible:
No
Triaged:
Yes
Groomed:
No
Sprint Candidate:
No
Tags:
QA Contact:
Complexity:
Smash Test:
Verified:
No
Verification Required:
No
Sprint:
Sprint 47

Description

Both bulk_create() and save() need to be called within a nested transaction so that it can be safely used in an outer transaction. Even though an IntegrityError is caught, it will still cause the transaction to be in an ERROR state and can no longer be used.

Associated revisions

Revision f4eb840b View on GitHub
Added by jortel@redhat.com 9 months ago

Fix BulkCreateManager.bulk_get_or_create()
closes #4306

Revision f4eb840b View on GitHub
Added by jortel@redhat.com 9 months ago

Fix BulkCreateManager.bulk_get_or_create()
closes #4306

History

#1 Updated by dkliban@redhat.com 9 months ago

  • Triaged changed from No to Yes
  • Sprint set to Sprint 47

#2 Updated by jortel@redhat.com 9 months ago

  • Status changed from NEW to ASSIGNED
  • Assignee set to jortel@redhat.com
  • Tags Pulp 3 RC Blocker added

#3 Updated by jortel@redhat.com 9 months ago

  • Status changed from ASSIGNED to POST

#4 Updated by jortel@redhat.com 9 months ago

  • Status changed from POST to MODIFIED

#5 Updated by daviddavis 8 months ago

  • Tags Pulp 3 added
  • Tags deleted (Pulp 3 RC Blocker)

#6 Updated by daviddavis 6 months ago

  • Sprint/Milestone set to 3.0

#7 Updated by bmbouter 6 months ago

  • Tags deleted (Pulp 3)

Please register to edit this issue

Also available in: Atom PDF