search

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

Only call sync for prepared statements remove sync message for internal errors #48

Closed alexr-bq closed 2 years ago

alexr-bq commented 2 years ago

The extra sync call previously added was only needed for prepared statements, and was being called in all cases instead, resulting in receiving two readyForQuery messages.

This PR also has the sync only sent for server errors now, and handles all internal driver errors by setting the readyForQuery flag instead.