DDC-2921: [GH-912] Avoid PersistentCollection::isEmpty() to fully load the collection. #3638

Closed
opened 2026-01-22 14:24:23 +01:00 by admin · 2 comments
Owner

Originally created by @doctrinebot on GitHub (Jan 17, 2014).

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 BenMorel:

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

Message:

This is a suggested simple performance improvement for PersistentCollection.

At the moment, isEmpty() always fully loads the collection. By changing the code to rely on count(), which makes advantage of extra lazy fetch when the collection is not loaded, this saves the overhead of loading the entire collection when one just wants to check whether the collection is empty.

Originally created by @doctrinebot on GitHub (Jan 17, 2014). 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 BenMorel: Url: https://github.com/doctrine/doctrine2/pull/912 Message: This is a suggested simple performance improvement for `PersistentCollection`. At the moment, `isEmpty()` always fully loads the collection. By changing the code to rely on `count()`, which makes advantage of extra lazy fetch when the collection is not loaded, this saves the overhead of loading the entire collection when one just wants to check whether the collection is empty.
admin added the Bug label 2026-01-22 14:24:23 +01:00
admin closed this issue 2026-01-22 14:24:23 +01:00
Author
Owner

@doctrinebot commented on GitHub (Feb 8, 2014):

Comment created by @doctrinebot:

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

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

@doctrinebot commented on GitHub (Feb 8, 2014):

Issue was closed with resolution "Fixed"

@doctrinebot commented on GitHub (Feb 8, 2014): Issue was closed with resolution "Fixed"
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: doctrine/archived-orm#3638