-
Notifications
You must be signed in to change notification settings - Fork 8.7k
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
optimize: upgrade alibaba/druid version to 1.2.20 #6523
Conversation
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.
Can you fix the issues exposed in these CI?
@funky-eyes 暂时先升级到1.2.20也能满足业务,升级到1.2.21和1.2.22遇到的CI问题我暂时不知道怎么解决:) |
请问这个pr可以合入到2.1.0版本嘛 @funky-eyes |
合并至2.2 |
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
Ⅰ. Describe what this PR did
将 alibaba/druid的版本从1.2.7升级到1.2.20
upgrade alibaba/druid version from 1.2.7 to 1.2.20
Ⅱ. Does this pull request fix one issue?
fixes #6522
Ⅲ. Why don't you add test cases (unit test/integration test)?
Ⅳ. Describe how to verify it
Ⅴ. Special notes for reviews