[PR #9935] Removed orderBy clause in walk select statement #11984

New Issue

admin · 2026-01-22T16:12:32+01:00

admin commented

2026-01-22 16:12:32 +01:00

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

State: closed
Merged: No

I've encountered an issue with an IBM Informix database
system would suffer heavily from having an order by in the
count select statement. Currently, this file contains a condition
where it will remove the order by statement for Sql Server.
Why are we keeping for other database systems? Even if it should
only make a minor difference, i can't see a reason to keep this
in the query since we are simply doing a count.

**Original Pull Request:** https://github.com/doctrine/orm/pull/9935 **State:** closed **Merged:** No --- I've encountered an issue with an IBM Informix database system would suffer heavily from having an order by in the count select statement. Currently, this file contains a condition where it will remove the order by statement for Sql Server. Why are we keeping for other database systems? Even if it should only make a minor difference, i can't see a reason to keep this in the query since we are simply doing a count.

admin added the pull-request label 2026-01-22 16:12:32 +01:00

admin referenced this issue

2026-01-22 16:16:57 +01:00

[PR #11984] Merge 2.20.x up into 3.3.x #13380

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#11984