pan3793 commented on code in PR #5907:
URL: https://github.com/apache/kyuubi/pull/5907#discussion_r1435033319
##########
externals/kyuubi-hive-sql-engine/src/main/scala/org/apache/kyuubi/engine/hive/HiveSQLEngine.scala:
##########
@@ -79,6 +79,14 @@ object HiveSQLEngine extends Logging {
kyuubiConf.setIfMissing(KyuubiConf.FRONTEND_THRIFT_BINARY_BIND_PORT, 0)
kyuubiConf.setIfMissing(HA_ZK_CONN_RETRY_POLICY,
RetryPolicies.N_TIME.toString)
+ // align with the operational behavior of HiveServer2, it is necessary to
+ // include the `hiveserver2-site.xml` configuration within the HiveConf
settings.
+ // for instance, upon the installation of the Hive Ranger plugin,
authorization
+ // configurations are appended to the `hiveserver2-site.xml` file.
Similarly, to activate
+ // the Ranger plugin for the Hive engine within Kyuubi, it is essential
for the Hive engine
+ // to load the `hiveserver2-site.xml` file. This ensures that the Hive
engine's
+ // security features are consistent with those managed by HiveServer2. see
KYUUBI-5878.
Review Comment:
```suggestion
// security features are consistent with those managed by HiveServer2.
See [KYUUBI #5878].
```
--
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]
