Doctrine 2 - ORM

JTI Owning table for identifier columns could/should be the entitytable

Details

  • Type: Improvement Improvement
  • Status: Open Open
  • Priority: Major Major
  • Resolution: Unresolved
  • Affects Version/s: None
  • Fix Version/s: None
  • Component/s: None
  • Security Level: All
  • Labels:
    None

Description

When ordering a JTI entity on id, the generated SQL will use the table of the root entity. This is because the root entity is listed as owner of the field in the _owningTableMap, leading to non-optimal queries.

More information see:
https://groups.google.com/forum/#!topic/doctrine-user/znkkP7IF_Aw

Activity

Ascending order - Click to sort in descending order
Hide
Permalink
Alexander added a comment -

I can pick this up if it's agreed upon that this could indeed be improved.

Show
Alexander added a comment - I can pick this up if it's agreed upon that this could indeed be improved.
Hide
Permalink
Marco Pivetta added a comment - - edited

The problem here is that joining with a JTI causes LEFT JOINS, which don't perform very well when it comes to sorting the results.

Just as a quick reference, here's where "something" should be changed to get this working:
https://github.com/doctrine/doctrine2/blob/master/lib/Doctrine/ORM/Query/SqlWalker.php#L316

When the field is part of the primary key, the field used for sorting results should be the one of the table of the entity itself, and not of the root of the CTI.

Show
Marco Pivetta added a comment - - edited The problem here is that joining with a JTI causes LEFT JOINS, which don't perform very well when it comes to sorting the results. Just as a quick reference, here's where "something" should be changed to get this working: https://github.com/doctrine/doctrine2/blob/master/lib/Doctrine/ORM/Query/SqlWalker.php#L316 When the field is part of the primary key, the field used for sorting results should be the one of the table of the entity itself, and not of the root of the CTI.

People

  • Assignee:
    Alexander
    Reporter:
    Alexander
Vote (0)
Watch (1)

Dates

  • Created:
    Updated: