[AMORO-3309][Improvement] Support for terminal integration with LDAP authentication in Kyuubi

[Aireed]

Why are the changes needed?

Close #xxx.
Support for terminal integration with LDAP authentication in Kyuubi

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)

[baiyangtx]

When LDAP is enabled, how is authentication information passed to Kyuubi?

[Aireed]

When LDAP is enabled, how is authentication information passed to Kyuubi?

Referring to this configuration manual, you can directly include the username and password in the URL

[tcodehuber]

@Aireed Can you also improve the helm charts? This can make us enable this feature easily. refs: https://github.com/apache/amoro/blob/master/charts/amoro/templates/amoro-configmap.yaml
https://github.com/apache/amoro/blob/master/charts/amoro/values.yaml

[Aireed]

@Aireed Can you also improve the helm charts? This can make us enable this feature easily. refs: https://github.com/apache/amoro/blob/master/charts/amoro/templates/amoro-configmap.yaml https://github.com/apache/amoro/blob/master/charts/amoro/values.yaml

I have updated the commition, PTAL. 3ks

[czy006]

LGTM

[codecov-commenter]

Codecov Report

Attention: Patch coverage is 0% with 7 lines in your changes missing coverage. Please review.

Project coverage is 30.16%. Comparing base (f13f3f3) to head (5bf9a66).
Report is 4 commits behind head on master.

Files with missing lines	Patch %	Lines
.../terminal/kyuubi/KyuubiTerminalSessionFactory.java	0.00%	7 Missing ⚠️

Additional details and impacted files

@@             Coverage Diff              @@
##             master    #3309      +/-   ##
============================================
- Coverage     30.17%   30.16%   -0.02%     
  Complexity     3840     3840              
============================================
  Files           580      580              
  Lines         48020    48037      +17     
  Branches       6207     6210       +3     
============================================
  Hits          14488    14488              
- Misses        32542    32559      +17     
  Partials        990      990              

Flag	Coverage Δ
core	30.16% <0.00%> (-0.02%)	⬇️

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]

Thanks for the contribution!