DDC-2790: [GH-845] Don't compute changeset for entities that are going to be deleted #3489

Closed
opened 2026-01-22 14:20:50 +01:00 by admin · 3 comments
Owner

Originally created by @doctrinebot on GitHub (Nov 11, 2013).

Originally assigned to: @beberlei on GitHub.

Jira issue originally created by user @doctrinebot:

This issue is created automatically through a Github pull request on behalf of flack:

Url: https://github.com/doctrine/doctrine2/pull/845

Message:

This is somewhat related to

http://www.doctrine-project.org/jira/browse/DDC-2761

I'm not sure if it fixes that particular problem, but it certainly helps with some unrelated issues I'm seeing in my code. My theory is that doctrine doesn't even need to compute changesets when an entity is scheduled to be removed, because all that would accomplish is to insert an UPDATE statement immediately before the DELETE statement for the entity.

The test suite passes, but I have not added a test specifically for this change (mostly because I wouldn't know exactly what to test). I could try to create one if necessary, though

Originally created by @doctrinebot on GitHub (Nov 11, 2013). Originally assigned to: @beberlei on GitHub. Jira issue originally created by user @doctrinebot: This issue is created automatically through a Github pull request on behalf of flack: Url: https://github.com/doctrine/doctrine2/pull/845 Message: This is somewhat related to http://www.doctrine-project.org/jira/browse/[DDC-2761](http://www.doctrine-project.org/jira/browse/DDC-2761) I'm not sure if it fixes that particular problem, but it certainly helps with some unrelated issues I'm seeing in my code. My theory is that doctrine doesn't even need to compute changesets when an entity is scheduled to be removed, because all that would accomplish is to insert an UPDATE statement immediately before the DELETE statement for the entity. The test suite passes, but I have not added a test specifically for this change (mostly because I wouldn't know exactly what to test). I could try to create one if necessary, though
admin added the Bug label 2026-01-22 14:20:50 +01:00
admin closed this issue 2026-01-22 14:20:52 +01:00
Author
Owner

@doctrinebot commented on GitHub (Jan 2, 2014):

Comment created by @doctrinebot:

A related Github Pull-Request [GH-845] was closed:
https://github.com/doctrine/doctrine2/pull/845

@doctrinebot commented on GitHub (Jan 2, 2014): Comment created by @doctrinebot: A related Github Pull-Request [GH-845] was closed: https://github.com/doctrine/doctrine2/pull/845
Author
Owner

@doctrinebot commented on GitHub (Jan 2, 2014):

Issue was closed with resolution "Fixed"

@doctrinebot commented on GitHub (Jan 2, 2014): Issue was closed with resolution "Fixed"
Author
Owner

@doctrinebot commented on GitHub (Jan 2, 2014):

Comment created by @doctrinebot:

A related Github Pull-Request [GH-829] was closed:
https://github.com/doctrine/doctrine2/pull/829

@doctrinebot commented on GitHub (Jan 2, 2014): Comment created by @doctrinebot: A related Github Pull-Request [GH-829] was closed: https://github.com/doctrine/doctrine2/pull/829
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: doctrine/archived-orm#3489