add a new config property authentication.ldap.guidKey

[lordk911]

for fix issues : #709

my ldap env use cn but uid for login, uid is the defaut behavior by kyuubi :

then try to connect kyuubi :

change kyuubi behavior to use cn:

then try to connect kyuubi :

[ulysses-you]

thank you for making a PR @lordk911 , welcome to Kyuubi !

[codecov-commenter]

Codecov Report

Merging #712 (0ab1a5b) into branch-1.2 (a6ab236) will increase coverage by 0.06%.
The diff coverage is 100.00%.

@@              Coverage Diff               @@
##           branch-1.2     #712      +/-   ##
==============================================
+ Coverage       80.11%   80.18%   +0.06%     
==============================================
  Files             120      120              
  Lines            4672     4678       +6     
  Branches          563      563              
==============================================
+ Hits             3743     3751       +8     
+ Misses            608      607       -1     
+ Partials          321      320       -1     

Impacted Files	Coverage Δ
...in/scala/org/apache/kyuubi/config/KyuubiConf.scala	94.08% <100.00%> (+0.08%)	⬆️
...uthentication/LdapAuthenticationProviderImpl.scala	92.00% <100.00%> (+0.33%)	⬆️
...org/apache/spark/kyuubi/SQLOperationListener.scala	97.43% <0.00%> (+5.12%)	⬆️

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update a6ab236...0ab1a5b. Read the comment docs.

[ulysses-you]

hi @lordk911 just notice this PR is for branch-1.2, can you change the target to master ? thanks

[yaooqinn]

It's better to add kshen@gaojihealth.com to your github settings too @lordk911