-
Notifications
You must be signed in to change notification settings - Fork 36
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
feat: added support for mysql2 v3.11.5 #1467
Conversation
ef485da
to
62635ca
Compare
Skipped the Couchbase update from 4.4.3 to 4.4.4, as it is addressing in the PR #1466 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM, waiting for the test to pass
62635ca
to
9f2dec5
Compare
ece2006
to
08fb03c
Compare
08fb03c
to
3b2d257
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Could you please add multi testing for "3.11.4" and "^3.11.5"?
3b2d257
to
eb1b1b6
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
One issue needs to be fixed. Otherwise pre-approve. LGTM!
Please use:
feat(mysql2): added support for v3.11.5 or feat: added support for mysql2 v3.11.5 |
In version 3.11.5 of the
mysql2
library, some changes were introduced in the internal structure of the library. Specifically:query
andexecute
methods were moved toBaseConnection
inlib/base/connection.js
.getConnection
,query
, andexecute
methods were relocated toBasePool
inlib/base/pool.js
.Updated instrumentation logic to:
Connection
prototype derives fromBaseConnection
for newer versions (v3.11.5+).Pool
class with changes to theBasePool
class.ref INSTA-20850
reference PR sidorares/node-mysql2#3081