This is an automated email from the ASF dual-hosted git repository.
benjobs pushed a commit to branch dev-2.1.5
in repository https://gitbox.apache.org/repos/asf/incubator-streampark.git
The following commit(s) were added to refs/heads/dev-2.1.5 by this push:
new 33249e6c0 [Improve] app start-modal minor improvement
33249e6c0 is described below
commit 33249e6c08d4ea7f416589fe22b0c3a693b55768
Author: benjobs <[email protected]>
AuthorDate: Mon Sep 16 12:30:07 2024 +0800
[Improve] app start-modal minor improvement
---
.../app/components/AppView/StartApplicationModal.vue | 2 +-
.../src/views/flink/app/hooks/useAppTableAction.ts | 13 +++++++++++++
.../flink/client/impl/KubernetesNativeSessionClient.scala | 2 +-
.../streampark/flink/client/impl/YarnSessionClient.scala | 3 ++-
.../flink/client/tool/FlinkSessionClientHelper.scala | 15 +++++++++++++++
.../streampark/flink/client/trait/FlinkClientTrait.scala | 14 --------------
6 files changed, 32 insertions(+), 17 deletions(-)
diff --git
a/streampark-console/streampark-console-webapp/src/views/flink/app/components/AppView/StartApplicationModal.vue
b/streampark-console/streampark-console-webapp/src/views/flink/app/components/AppView/StartApplicationModal.vue
index 82ed8bc2e..9c415fd95 100644
---
a/streampark-console/streampark-console-webapp/src/views/flink/app/components/AppView/StartApplicationModal.vue
+++
b/streampark-console/streampark-console-webapp/src/views/flink/app/components/AppView/StartApplicationModal.vue
@@ -75,7 +75,7 @@
checkedChildren: 'ON',
unCheckedChildren: 'OFF',
},
- defaultValue: receiveData.historySavePoint &&
receiveData.historySavePoint.length > 0,
+ defaultValue: receiveData.selected != null,
afterItem: () => h('span', { class: 'pop-tip' },
t('flink.app.view.savepointTip')),
},
{
diff --git
a/streampark-console/streampark-console-webapp/src/views/flink/app/hooks/useAppTableAction.ts
b/streampark-console/streampark-console-webapp/src/views/flink/app/hooks/useAppTableAction.ts
index c83e29895..6956e69c4 100644
---
a/streampark-console/streampark-console-webapp/src/views/flink/app/hooks/useAppTableAction.ts
+++
b/streampark-console/streampark-console-webapp/src/views/flink/app/hooks/useAppTableAction.ts
@@ -64,6 +64,12 @@ export const useAppTableAction = (
/* Operation button list */
function getActionList(record: AppListRecord, currentPageNo: number):
ActionItem[] {
return [
+ {
+ tooltip: { title: t('flink.app.operation.detail') },
+ auth: 'app:detail',
+ icon: 'carbon:data-view-alt',
+ onClick: handleDetail.bind(null, record),
+ },
{
tooltip: { title: t('flink.app.operation.release') },
ifShow:
@@ -210,6 +216,13 @@ export const useAppTableAction = (
}),
};
}
+
+ /* Click for details */
+ function handleDetail(app: AppListRecord) {
+ flinkAppStore.setApplicationId(app.id);
+ router.push({ path: '/flink/app/detail', query: { appId: app.id } });
+ }
+
/* Click to edit */
function handleEdit(app: AppListRecord, currentPageNo: number) {
// Record the current page number
diff --git
a/streampark-flink/streampark-flink-client/streampark-flink-client-core/src/main/scala/org/apache/streampark/flink/client/impl/KubernetesNativeSessionClient.scala
b/streampark-flink/streampark-flink-client/streampark-flink-client-core/src/main/scala/org/apache/streampark/flink/client/impl/KubernetesNativeSessionClient.scala
index 4c3cf5712..f9ea57cda 100644
---
a/streampark-flink/streampark-flink-client/streampark-flink-client-core/src/main/scala/org/apache/streampark/flink/client/impl/KubernetesNativeSessionClient.scala
+++
b/streampark-flink/streampark-flink-client/streampark-flink-client-core/src/main/scala/org/apache/streampark/flink/client/impl/KubernetesNativeSessionClient.scala
@@ -151,7 +151,7 @@ object KubernetesNativeSessionClient extends
KubernetesNativeClientTrait with Lo
val flinkConfig = getFlinkK8sConfig(deployRequest)
- replaceConfig(flinkConfig, "\\$\\{job(Name|name)}|\\$job(Name|name)",
deployRequest.clusterName)
+ FlinkSessionSubmitHelper.doReplaceJobName(flinkConfig,
deployRequest.clusterName)
val kubeClient =
FlinkKubeClientFactory.getInstance.fromConfiguration(flinkConfig, "client")
diff --git
a/streampark-flink/streampark-flink-client/streampark-flink-client-core/src/main/scala/org/apache/streampark/flink/client/impl/YarnSessionClient.scala
b/streampark-flink/streampark-flink-client/streampark-flink-client-core/src/main/scala/org/apache/streampark/flink/client/impl/YarnSessionClient.scala
index 76d86b8b6..091e6a998 100644
---
a/streampark-flink/streampark-flink-client/streampark-flink-client-core/src/main/scala/org/apache/streampark/flink/client/impl/YarnSessionClient.scala
+++
b/streampark-flink/streampark-flink-client/streampark-flink-client-core/src/main/scala/org/apache/streampark/flink/client/impl/YarnSessionClient.scala
@@ -20,6 +20,7 @@ package org.apache.streampark.flink.client.impl
import org.apache.streampark.common.util.Utils
import org.apache.streampark.flink.client.`trait`.YarnClientTrait
import org.apache.streampark.flink.client.bean._
+import org.apache.streampark.flink.client.tool.FlinkSessionSubmitHelper
import org.apache.commons.lang3.StringUtils
import org.apache.flink.client.deployment.ClusterSpecification
@@ -77,7 +78,7 @@ object YarnSessionClient extends YarnClientTrait {
// app name
.safeSet(YarnConfigOptions.APPLICATION_NAME, deployRequest.clusterName)
- replaceConfig(flinkConfig, "\\$\\{job(Name|name)}|\\$job(Name|name)",
deployRequest.clusterName)
+ FlinkSessionSubmitHelper.doReplaceJobName(flinkConfig,
deployRequest.clusterName)
logInfo(s"""
|------------------------------------------------------------------
diff --git
a/streampark-flink/streampark-flink-client/streampark-flink-client-core/src/main/scala/org/apache/streampark/flink/client/tool/FlinkSessionClientHelper.scala
b/streampark-flink/streampark-flink-client/streampark-flink-client-core/src/main/scala/org/apache/streampark/flink/client/tool/FlinkSessionClientHelper.scala
index 2e1945e96..fcfcd3fc1 100644
---
a/streampark-flink/streampark-flink-client/streampark-flink-client-core/src/main/scala/org/apache/streampark/flink/client/tool/FlinkSessionClientHelper.scala
+++
b/streampark-flink/streampark-flink-client/streampark-flink-client-core/src/main/scala/org/apache/streampark/flink/client/tool/FlinkSessionClientHelper.scala
@@ -34,6 +34,7 @@ import org.json4s.jackson.Serialization
import java.io.File
import java.nio.charset.StandardCharsets
+import scala.collection.JavaConversions._
import scala.util.{Failure, Success, Try}
object FlinkSessionSubmitHelper extends Logger {
@@ -102,6 +103,20 @@ object FlinkSessionSubmitHelper extends Logger {
}
}
+ private[client] def doReplaceJobName(flinkConfig: Configuration,
replacement: String): Unit = {
+ flinkConfig
+ .keySet()
+ .foreach(
+ k => {
+ val v = flinkConfig.getString(k, null)
+ if (v != null) {
+ val result = v
+ .replaceAll("\\$\\{job(Name|name)}|\\$job(Name|name)",
replacement)
+ flinkConfig.setString(k, result)
+ }
+ })
+ }
+
}
/**
diff --git
a/streampark-flink/streampark-flink-client/streampark-flink-client-core/src/main/scala/org/apache/streampark/flink/client/trait/FlinkClientTrait.scala
b/streampark-flink/streampark-flink-client/streampark-flink-client-core/src/main/scala/org/apache/streampark/flink/client/trait/FlinkClientTrait.scala
index d741df50f..896767ca3 100644
---
a/streampark-flink/streampark-flink-client/streampark-flink-client-core/src/main/scala/org/apache/streampark/flink/client/trait/FlinkClientTrait.scala
+++
b/streampark-flink/streampark-flink-client/streampark-flink-client-core/src/main/scala/org/apache/streampark/flink/client/trait/FlinkClientTrait.scala
@@ -496,18 +496,4 @@ trait FlinkClientTrait extends Logger {
})
}
- def replaceConfig(flinkConfig: Configuration, regexp: String, replacement:
String): Unit = {
- flinkConfig
- .keySet()
- .foreach(
- k => {
- val v = flinkConfig.getString(k, null)
- if (v != null) {
- val result = v
- .replaceAll(regexp, replacement)
- flinkConfig.setString(k, result)
- }
- })
- }
-
}