Skip to content

Conversation

Tgarifullin
Copy link

Добрый день, спасибо за библиотеку.
Добавил для возможности доступа к напрямую к курлеру Client->transport()->getCurler() (иногда бывает нужно).

@renatko
Copy link

renatko commented Sep 7, 2016

Как насчет того, чтобы использовать Guzzle вместо своей обёртки над curl?

@isublimity isublimity merged commit 1612fc7 into smi2:master Sep 8, 2016
@isublimity
Copy link
Contributor

Добрый день, а можете показать пример когда нужно получить доступ к Curler ? Просто интересно с чем вы столкнулись - что не заложенно в драйвер, или это специфичный кейс ?

Как насчет того, чтобы использовать Guzzle вместо своей обёртки над curl?

Нет, мы не будем переводить код на Guzzle, из за специфики использования в проектах.

@Tgarifullin
Copy link
Author

Случай скорее специфичный - в проекте на yii1 при заворачивании Client в CDBConnection для возможности доступа CDBCommand к вложенному Curler.
Возможно стоит добавить в драйвер довольно опасный метод, что-то наподобие executeSQL который бы давал возможность выполнить произвольный sql без какой-либо обработки.

@renatko
Copy link

renatko commented Sep 8, 2016

Возможно стоит добавить в драйвер довольно опасный метод, что-то наподобие executeSQL который бы давал возможность выполнить произвольный sql без какой-либо обработки.

я использую $clickhouse->transport()->write($sql);
где $clickhouse - инстанс \ClickHouseDB\Client

@isublimity
Copy link
Contributor

Я думаю сделаю так : #5
Мы отказались от Conditions, в часте проектов и он скоро будет не нужен

@isublimity
Copy link
Contributor

isublimity commented Sep 17, 2016

Дополню ответ почему не Guzzle и не psr ,
https://github.com/smi2/phpClickHouse/blob/cluster/doc/article_ru_01.md#Установка
;)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants