cxzl25 commented on a change in pull request #2165:
URL: https://github.com/apache/incubator-kyuubi/pull/2165#discussion_r829694666



##########
File path: docs/security/hadoop_credentials_manager.md
##########
@@ -83,8 +83,8 @@ The necessary configurations for hdfs and hive to obtain 
delegation token are as
 
 Key | Meaning | value
 --- | --- | ---
-<div style='width: 40pt;word-wrap: break-word;white-space: 
normal'>hadoop.security.authentication</div>|<div style='width: 40pt;word-wrap: 
break-word;white-space: normal'>Set the authentication for the 
cluster</div>|<div style='width: 40pt;word-wrap: break-word;white-space: 
normal'>kerberos</div>
-<div style='width: 40pt;word-wrap: break-word;white-space: 
normal'>hive.metastore.uris</div>|<div style='width: 40pt;word-wrap: 
break-word;white-space: normal'>URI for client to contact metastore 
server</div>|<div style='width: 40pt;word-wrap: break-word;white-space: 
normal'>thrift://{metastoreHost}:{metastorePort}}</div>
-<div style='width: 40pt;word-wrap: break-word;white-space: 
normal'>hive.metastore.sasl.enabled</div>|<div style='width: 40pt;word-wrap: 
break-word;white-space: normal'>If true, the metastore thrift interface will be 
secured with SASL.Clients must authenticate with Kerberos.</div>|<div 
style='width: 40pt;word-wrap: break-word;white-space: normal'>true</div>
-<div style='width: 40pt;word-wrap: break-word;white-space: 
normal'>hive.metastore.kerberos.principal</div>|<div style='width: 
40pt;word-wrap: break-word;white-space: normal'>The service principal for the 
metastore thrift server. The special string _HOST will be replaced 
automatically with the correct host name.</div>|<div style='width: 
40pt;word-wrap: break-word;white-space: normal'>for example 
hive/_HOST@${realm}</div>
-<div style='width: 40pt;word-wrap: break-word;white-space: 
normal'>hive.metastore.kerberos.keytab.file</div>|<div style='width: 
40pt;word-wrap: break-word;white-space: normal'>The path to the Kerberos Keytab 
file containing the metastore thrift server's service principal.</div>|<div 
style='width: 40pt;word-wrap: break-word;white-space: normal'>for example 
/etc/security/keytabs/hive.service.keytab</div>
\ No newline at end of file
+hadoop\.security<br>\.authentication|<div style='width: 40pt;word-wrap: 
break-word;white-space: normal'>Set the authentication for the 
cluster</div>|<div style='width: 40pt;word-wrap: break-word;white-space: 
normal'>kerberos</div>

Review comment:
       > It seems that the previous part of this document is also not modified 
according to #1890
   
   Sorry I didn't notice this document when I changed it.
   There are two more.
   `authentication.md`
   `kinit.md`
   




-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]



---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to