search

keboola / php-db-import

MIT License
4 stars 0 forks source link

QueryTimeout paramater took me all day ... #58

Closed yustme closed 5 years ago

yustme commented 5 years ago

Tenhle parametr vůbec nedělá, to co by měl. Já bych ho odstranil a nechal ho zemřít hrozivou smrtí... :skull: :imp: https://github.com/keboola/php-db-import/blob/cd99cfb57ae7a9e3928390d96000d82a12d009ba/src/Snowflake/Connection.php#L27

Místo toho, aby ustřihnul spojení po XX vteřinách, tak počká jak to dopadne a pokud se přesáhne "zhruba" dobu timeoutu, tak jenom napíše, že se to nestihlo, ale dotaz doběhne do konce.

ErikZigo commented 5 years ago

Ony obecne ty jejich DSN parametry jsou sranda a divna vec.

yustme commented 5 years ago

Problém byl, že jsem to testoval na funkci sleep, která se chová jinak než jiné query.