turboFei commented on code in PR #7041:
URL: https://github.com/apache/kyuubi/pull/7041#discussion_r2056973835
##########
kyuubi-server/src/main/scala/org/apache/kyuubi/session/KyuubiSessionManager.scala:
##########
@@ -52,12 +52,8 @@ class KyuubiSessionManager private (name: String) extends
SessionManager(name) {
val operationManager = new KyuubiOperationManager()
val credentialsManager = new HadoopCredentialsManager()
- val applicationManager = new KyuubiApplicationManager()
-
- // Currently, the metadata manager is used by the REST frontend which
provides batch job APIs,
- // so we initialize it only when Kyuubi starts with the REST frontend.
- lazy val metadataManager: Option[MetadataManager] =
- if (conf.isRESTEnabled) Some(new MetadataManager()) else None
+ val metadataManager: MetadataManager = new MetadataManager()
Review Comment:
It is fine to always create metadatamanager, especially we are going to
persist the kubernetes information into database.
--
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: notifications-unsubscr...@kyuubi.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: notifications-unsubscr...@kyuubi.apache.org
For additional commands, e-mail: notifications-h...@kyuubi.apache.org
