Skip to content

Conversation

cmoog
Copy link
Owner

@cmoog cmoog commented Mar 23, 2022

Closes #26

@cmoog cmoog merged commit e955f6c into master Mar 23, 2022
@cmoog cmoog deleted the querytimeout branch March 23, 2022 20:44
@RollsChris
Copy link

Hi.. is there a reason why this was not applied to the other drivers?

@cmoog
Copy link
Owner Author

cmoog commented Mar 23, 2022

@RedEyedDog
The query timeout is passed through to all three drivers currently supported: mysql, postgres, and mssql.

The relevant code snippets:

timeout: queryTimeout,

query_timeout: queryTimeout,

requestTimeout: config.queryTimeout,

At the moment, none of the driver behavior is covered by automated tests, so please do open an issue if something isn't working as expected.

@RollsChris
Copy link

hey thanks, i didn't notice the others ;) do you know when this will be release its bugging me

@cmoog
Copy link
Owner Author

cmoog commented Mar 24, 2022

@RedEyedDog v0.5.0 is live!

@RollsChris
Copy link

@cmoog owww yes.. thanks :)

@RollsChris
Copy link

i can confirm this is working very nice for mssql

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.

Increase timeout

2 participants