В
Yuriy Mamaev
Хорошо, тогда разберу этот PR и подумаю как допилить.
Лучше поревьюить и оставить замечания, если будут. Да и в целом можно для начала в пакете целиком разобраться и понаписать issue про сомнительные места.
Size: a a a
2020 September 04
T
@viktorprogger it is possible to prepare a benchmark test for db.
YM
Лучше поревьюить и оставить замечания, если будут. Да и в целом можно для начала в пакете целиком разобраться и понаписать issue про сомнительные места.
ок, разберусь
В
TradersVE
@viktorprogger it is possible to prepare a benchmark test for db.
What do you mean?
В
Yuriy Mamaev
Хорошо, тогда разберу этот PR и подумаю как допилить.
А сам pr еще не допилен. Там много мутантов осталось)
NO
Не согласен. Писать вот такое в самом пакете не норм:
public function __construct(?DriverInterface $driver = null)
{
if ($driver === null) {
$driver = new RedisDriver('localhost:1010');
}
$this->driver = $driver;
}
public function __construct(?DriverInterface $driver = null)
{
if ($driver === null) {
$driver = new RedisDriver('localhost:1010');
}
$this->driver = $driver;
}
Да, подобные умолчания могут задаваться только на уровне приложения. Видимо мы о разных умолчаниях.
В
TradersVE
@viktorprogger it is possible to prepare a benchmark test for db.
what exactly do you want to benchmark?
AM
Да, подобные умолчания могут задаваться только на уровне приложения. Видимо мы о разных умолчаниях.
О каких умолчаниях ты?
T
what exactly do you want to benchmark?
I'm going to finish uploading the coverage to 100%, when I'm ready I'll let you know what I want to test.
NO
Ну всякие мелочи конфигурируемые. Размер буфера в логгере и т.д.
AM
Ну всякие мелочи конфигурируемые. Размер буфера в логгере и т.д.
Ну так кроме них ещё дофига всего.
СП
Там же во всех вариантах разные строки засчет пробелов и Тамбов и переносов
AM
Там же во всех вариантах разные строки засчет пробелов и Тамбов и переносов
Хоть кто-то заметил :)
NO
обижаешь (
NO
это сразу видно
В
TradersVE
I'm going to finish uploading the coverage to 100%, when I'm ready I'll let you know what I want to test.
Ok, I'll wait for your message
СП
done
Благодарю! :)