Use php-ds <3 #5667

Open
opened 2026-01-22 15:14:09 +01:00 by admin · 8 comments
Owner

Originally created by @guilhermeblanco on GitHub (Sep 2, 2017).

Originally created by @guilhermeblanco on GitHub (Sep 2, 2017).
Author
Owner

@Anyqax commented on GitHub (Sep 2, 2017):

Just curious... does that imply everyone would have to install the php-ds extension (or suffer the performance penalties of the polyfill)? Are there specific parts of the orm which are expected to perform better with the extension?

@Anyqax commented on GitHub (Sep 2, 2017): Just curious... does that imply everyone would have to install the php-ds extension (or suffer the performance penalties of the polyfill)? Are there specific parts of the orm which are expected to perform better with the extension?
Author
Owner

@Ocramius commented on GitHub (Sep 2, 2017):

Are there specific parts of the orm which are expected to perform better with the extension?

The UnitOfWork would benefit massively from an object-to-object map

@Ocramius commented on GitHub (Sep 2, 2017): > Are there specific parts of the orm which are expected to perform better with the extension? The `UnitOfWork` would benefit massively from an object-to-object map
Author
Owner

@Anyqax commented on GitHub (Sep 2, 2017):

Ah okay, excited to see how 3.0 turns out. Thanks for your efforts!

@Anyqax commented on GitHub (Sep 2, 2017): Ah okay, excited to see how 3.0 turns out. Thanks for your efforts!
Author
Owner

@Majkl578 commented on GitHub (Sep 2, 2017):

The UnitOfWork would benefit massively from an object-to-object map

Object-to-object is possible with SPL's SplObjectStorage already.

Last time I was playing with php-ds, it didn't convince me with stability, many crashes, SIGBUSes etc... 😕

@Majkl578 commented on GitHub (Sep 2, 2017): > The UnitOfWork would benefit massively from an object-to-object map Object-to-object is possible with SPL's SplObjectStorage already. Last time I was playing with php-ds, it didn't convince me with stability, many crashes, SIGBUSes etc... 😕
Author
Owner

@Ocramius commented on GitHub (Sep 2, 2017):

it didn't convince me with stability

It can also get better :-P

@Ocramius commented on GitHub (Sep 2, 2017): > it didn't convince me with stability It can also get better :-P
Author
Owner

@alsar commented on GitHub (Apr 4, 2018):

Is this still an option for 3.0?

@alsar commented on GitHub (Apr 4, 2018): Is this still an option for 3.0?
Author
Owner

@Ocramius commented on GitHub (Apr 4, 2018):

@alsar only if benchmarks show an actual improvement.

@Ocramius commented on GitHub (Apr 4, 2018): @alsar only if benchmarks show an actual improvement.
Author
Owner

@glensc commented on GitHub (Aug 31, 2019):

nice ticket id #6666 :)

@glensc commented on GitHub (Aug 31, 2019): nice ticket id #6666 :)
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: doctrine/archived-orm#5667