search

dvizh / yii2-shop

Admin panel for a shop based on Yii2
Apache License 2.0
32 stars 20 forks source link

Optimized recursion #21

Closed Deadly299 closed 6 years ago

pistol88 commented 6 years ago

Почему через createCommand?

Deadly299 commented 6 years ago

В угоду скорости

pistol88 commented 6 years ago

При этом теряется даже теоретическая поддержка баз данных, отличных от mysql. Замени на AR. Если нужна производительность, добавь ->asArray, тогда это будет работать почти также быстро и экономно, как твой вариант.

Deadly299 commented 6 years ago

Про поддержку действительно почему-то не подумал

pistol88 commented 6 years ago

Изменено кол-во аргументов и их название, то есть это мажорное изменение, которое сломает все работающие проекты, использующие этот метод. Думаю, ты помнишь реакцию сообщества на такие изменения и "подрывы" после них (нас могут вычислить по ИП).

Может, сможешь оптимизировать без изменения аргументов?

Deadly299 commented 6 years ago

Сделал. Русские не сдаются!!!