[AMORO-2888] Can't establish connections when rebooting ams

[tcodehuber]

Why are the changes needed?

Close #2888.

Brief change log

How was this patch tested?

• Add some test cases that check the changes thoroughly including negative and positive cases if possible

• Add screenshots for manual tests if appropriate

• Run test locally before making a pull request

Documentation

• Does this pull request introduce a new feature? (yes / no)
• If yes, how is the feature documented? (not applicable / docs / JavaDocs / not documented)

[zhoujinsong]

@tcodehuber Thanks for the contribution!

But I am not sure if allowPublicKeyRetrieval=true is a common property to all MySQL versions.

[tcodehuber]

allowPublicKeyRetrieval=true

Although it's imported by mysql8, adding it in the con url will not affect other editions of mysql. It will be ignored if connecting to the mysql below version 8, which will not take side effort. I have tested that in my own env.

[codecov-commenter]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 27.56%. Comparing base (4e3e5c6) to head (3190af3).
Report is 225 commits behind head on master.

Additional details and impacted files

@@             Coverage Diff              @@
##             master    #2889      +/-   ##
============================================
+ Coverage     27.47%   27.56%   +0.08%     
  Complexity     2483     2483              
============================================
  Files           360      360              
  Lines         37213    37099     -114     
  Branches       5432     5367      -65     
============================================
+ Hits          10226    10227       +1     
+ Misses        26027    25912     -115     
  Partials        960      960              

Flag	Coverage Δ
trino	27.56% <ø> (+0.08%)	⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[zhoujinsong]

LGTM.

[zhoujinsong]

Although it's imported by mysql8, adding it in the con url will not affect other editions of mysql. It will be ignored if connecting to the mysql below version 8, which will not take side effort. I have tested that in my own env.

Thanks for the information.