Move Snowflake Connection to standalone package

keboola / php-db-import

MIT License

4 stars 0 forks source link

Move Snowflake Connection to standalone package #48

Open Halama opened 5 years ago

Halama commented 5 years ago

Vytáhnout Connection Class do samostatného balíčku.

pročistit ji
správně odchytávat odbc chyby a vyhazovat vlastní vyjímky
podoprovat správné bindování #45
ověřit funkčnost qouteIdentifier https://keboola.slack.com/archives/C0308KGPP/p1543316594029300
přidat možnost explicitně zavřít spojení https://github.com/keboola/connection/issues/1526

Halama commented 5 years ago

Případně varianta dohodnout se že budeme používat něco hotového Dibi, Zend DB 2?

Halama commented 5 years ago

Ještě jsem nedávno v nějaké diskuzi u výpadku SNFLK zahlédl že tam psal někdo kdo používá PDO pro SNFLK. Pořád to je ale v preview https://github.com/snowflakedb/pdo_snowflake Vypadá to ale že nějaká aktivita tam je.

tomasfejfar commented 4 years ago

Connection se používala už ve writeru, tady a v DWHM. Teď bych ji měl použít v Looker writeru, tak je asi správný čas to extrahovat a mít v extra package. Až bude hotová, tak bude potřeba to tady zrefaktorovat, aby to bylo zpětně kompatibiliní (případně vydat novou verzi a upravit to podle potřeby tam kde se to používá.