deliverr / data-knex-snowflake-dialect

knex.js dialect for the Snowflake data warehouse
MIT License
16 stars 26 forks source link

Connection caneclled check #51

Closed barooo closed 1 year ago

barooo commented 1 year ago

canceling a query can leave the connection in a bad state, but the validation method just checks presence so it would keep getting returned to the pool, eventually resulting in all connections being dead which leaves the app in a 503 state.

barooo commented 1 year ago

I accidentally did the wrong fork here, all of these changes will hopefully be ready to PR eventually but not yet.