CodeFoodPixels / node-promise-mysql

A wrapper for mysqljs/mysql that wraps function calls with Bluebird promises.
MIT License
338 stars 64 forks source link

Changed default error handler #107

Closed bogdan-panteleev closed 5 years ago

bogdan-panteleev commented 5 years ago

removed default console log of error and left it only for the case, when there is no reconnection after

CodeFoodPixels commented 5 years ago

Why?

bogdan-panteleev commented 5 years ago

Why?

In order not to have this big annoying message in cases, when reconnection is reestablished automatically. It should be shown only in critical cases

CodeFoodPixels commented 5 years ago

This would require a major version bump for something that I feel is unnecessary at this point. Closing.

CodeFoodPixels commented 5 years ago

I have a beta for the rewrite (v4.0.0) out at the moment that doesn't post these errors