[PR #476] Replaced the setting of Ids to the whereInQuery with a more performant implementation #8256

New Issue

admin · 2026-01-22T15:59:05+01:00

admin commented

2026-01-22 15:59:05 +01:00

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

State: closed
Merged: No

While working with large datasets in combination with pagination (ie. a pagination window of 100 or more), the performance drastically decreased.

I tracked the problem down to the setting of the parameters. In the old implementation, the parameters are set individually. This pull request sets the parameter as an array to the Query and Walker.

Testing results (# of entities, original performance, improved performance):
32 0,036 0,033
64 0,043 0,037
128 0,065 0,043
256 0,133 0,057
512 0,360 0,088
1024 1,212 0,157
2048 4,462 0,332
4096 17,203 0,818

**Original Pull Request:** https://github.com/doctrine/orm/pull/476 **State:** closed **Merged:** No --- While working with large datasets in combination with pagination (ie. a pagination window of 100 or more), the performance drastically decreased. I tracked the problem down to the setting of the parameters. In the old implementation, the parameters are set individually. This pull request sets the parameter as an array to the Query and Walker. Testing results (# of entities, original performance, improved performance): 32 0,036 0,033 64 0,043 0,037 128 0,065 0,043 256 0,133 0,057 512 0,360 0,088 1024 1,212 0,157 2048 4,462 0,332 4096 17,203 0,818

admin added the pull-request label 2026-01-22 15:59:05 +01:00

admin closed this issue

2026-01-22 15:59:05 +01:00

Sign in to join this conversation.

Branches Tags

1 Participants

Notifications

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: doctrine/archived-orm#8256