[PR #1033] Lazy criteria for ManyToMany collection #9071

New Issue

admin · 2026-01-22T16:03:04+01:00

admin commented

2026-01-22 16:03:04 +01:00

Original Pull Request: https://github.com/doctrine/orm/pull/1033

State: closed
Merged: No

This continues my previous work on making Criteria most efficient.

Currently we are wrapping matching calls on repositories and matching calls on EXTRA_LAZY associations around a LazyCriteria. However, ManyToMany are still completely loaded: https://github.com/doctrine/doctrine2/blob/master/lib/Doctrine/ORM/PersistentCollection.php#L874

This is still problematic from a performance point of view because count, contains... cannot be optimized. I think the solution is similar to previous one, hence creating a Lazy collection for that kind of associations.

However, this is really tricky to do because of the whole mess inside the persisters (can't wait for them to be completely refactored, it's getting really hard to maintain this mess :p).

**Original Pull Request:** https://github.com/doctrine/orm/pull/1033 **State:** closed **Merged:** No --- This continues my previous work on making Criteria most efficient. Currently we are wrapping matching calls on repositories and matching calls on EXTRA_LAZY associations around a LazyCriteria. However, ManyToMany are still completely loaded: https://github.com/doctrine/doctrine2/blob/master/lib/Doctrine/ORM/PersistentCollection.php#L874 This is still problematic from a performance point of view because count, contains... cannot be optimized. I think the solution is similar to previous one, hence creating a Lazy collection for that kind of associations. However, this is really tricky to do because of the whole mess inside the persisters (can't wait for them to be completely refactored, it's getting really hard to maintain this mess :p).

admin added the pull-request label 2026-01-22 16:03:04 +01:00

admin closed this issue

2026-01-22 16:03:04 +01:00

admin referenced this issue

2026-01-22 16:10:29 +01:00

[PR #9071] Merge up #11350

Sign in to join this conversation.