You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@flink.apache.org by xt...@apache.org on 2021/09/23 05:52:25 UTC
[flink] branch release-1.13 updated: [FLINK-24353][scripts] Respect
dynamic configurations when calculating memory sizes.
This is an automated email from the ASF dual-hosted git repository.
xtsong pushed a commit to branch release-1.13
in repository https://gitbox.apache.org/repos/asf/flink.git
The following commit(s) were added to refs/heads/release-1.13 by this push:
new b8c1a59 [FLINK-24353][scripts] Respect dynamic configurations when calculating memory sizes.
b8c1a59 is described below
commit b8c1a597533e2d9091caa8f3d864e24be4fb8ad2
Author: Xintong Song <to...@gmail.com>
AuthorDate: Wed Sep 22 16:35:38 2021 +0800
[FLINK-24353][scripts] Respect dynamic configurations when calculating memory sizes.
This closes #17335
---
flink-dist/src/main/flink-bin/bin/config.sh | 6 +++---
1 file changed, 3 insertions(+), 3 deletions(-)
diff --git a/flink-dist/src/main/flink-bin/bin/config.sh b/flink-dist/src/main/flink-bin/bin/config.sh
index a16db37..d3a8d78 100755
--- a/flink-dist/src/main/flink-bin/bin/config.sh
+++ b/flink-dist/src/main/flink-bin/bin/config.sh
@@ -526,7 +526,7 @@ extractLoggingOutputs() {
parseResourceParamsAndExportLogs() {
local cmd=$1
- java_utils_output=$(runBashJavaUtilsCmd ${cmd} "${FLINK_CONF_DIR}" "${FLINK_BIN_DIR}/bash-java-utils.jar:$(findFlinkDistJar)" "$@")
+ java_utils_output=$(runBashJavaUtilsCmd ${cmd} "${FLINK_CONF_DIR}" "${FLINK_BIN_DIR}/bash-java-utils.jar:$(findFlinkDistJar)" "${@:2}")
logging_output=$(extractLoggingOutputs "${java_utils_output}")
params_output=$(extractExecutionResults "${java_utils_output}" 2)
@@ -552,9 +552,9 @@ logs: $logging_output
}
parseJmArgsAndExportLogs() {
- parseResourceParamsAndExportLogs GET_JM_RESOURCE_PARAMS
+ parseResourceParamsAndExportLogs GET_JM_RESOURCE_PARAMS "$@"
}
parseTmArgsAndExportLogs() {
- parseResourceParamsAndExportLogs GET_TM_RESOURCE_PARAMS
+ parseResourceParamsAndExportLogs GET_TM_RESOURCE_PARAMS "$@"
}