Skip to content

Commit

Permalink
rename params to support multiple engines
Browse files Browse the repository at this point in the history
  • Loading branch information
zhaohehuhu committed Mar 15, 2024
1 parent 991f58a commit f54efdb
Showing 1 changed file with 6 additions and 6 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -249,9 +249,9 @@ private[v1] class AdminResource extends ApiRequestContext with Logging {
@QueryParam("hive.server2.proxy.user") hs2ProxyUser: String,
@QueryParam("forceKill") @DefaultValue("false") forceKill: Boolean,
@QueryParam("refId") refId: String,
@QueryParam("sparK8Context") sparK8Context: String,
@QueryParam("sparK8Namespace") sparK8Namespace: String,
@QueryParam("sparkMaster") sparkMaster: String): Response = {
@QueryParam("kubernetesContext") kubernetesContext: String,
@QueryParam("kubernetesNamespace") kubernetesNamespace: String,
@QueryParam("resourceManager") resourceManager: String): Response = {
val activeProxyUser = Option(kyuubiProxyUser).getOrElse(hs2ProxyUser)
val userName = if (fe.isAdministrator(fe.getRealUser())) {
Option(activeProxyUser).getOrElse(fe.getRealUser())
Expand All @@ -267,9 +267,9 @@ private[v1] class AdminResource extends ApiRequestContext with Logging {
}

val applicationManagerInfo = ApplicationManagerInfo(
Option(sparkMaster),
Option(sparK8Context),
Option(sparK8Namespace))
Option(resourceManager),
Option(kubernetesContext),
Option(kubernetesNamespace))

val killMessage = fe.be.sessionManager.asInstanceOf[KyuubiSessionManager]
.applicationManager.killApplication(applicationManagerInfo, refId)
Expand Down

0 comments on commit f54efdb

Please sign in to comment.