Update logic around error handling with prepared statements

vertica / vertica-nodejs

Official native node.js client for the Vertica Analytics Database.

https://www.vertica.com/

Apache License 2.0

13 stars 14 forks source link

Update logic around error handling with prepared statements #38

Closed alexr-bq closed 2 years ago

alexr-bq commented 2 years ago

There was an issue where we couldn't use callbacks and call a second query after a prepared statement failed. This is because sync() was never called in this case. This PR does two things:

Remove built-in ready for query flag in internal error cases and always sync after an error
Add new flag to make sure we only call the callback once in an error case