XComp commented on a change in pull request #13747:
URL: https://github.com/apache/flink/pull/13747#discussion_r512553693
##########
File path: flink-dist/src/main/flink-bin/bin/jobmanager.sh
##########
@@ -51,6 +51,10 @@ if [[ $STARTSTOP == "start" ]] || [[ $STARTSTOP ==
"start-foreground" ]]; then
args+=("--webui-port")
args+=("${WEBUIPORT}")
fi
+
+ if [ ! -z "${DYNAMIC_PARAMETERS+x}" ]; then
+ for v in ${DYNAMIC_PARAMETERS[@]}; do args+=($v); done
Review comment:
When I implemented it firstly, I ran into issues. Hence, I used the for
loop option. I addressed it again and it worked. No clue, what was wrong in the
first place. 🤔 Anyway, I fixed it. Now, the `args` array simply gets
concatenated with the `DYNAMIC_PARAMETERS` array.
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
[email protected]
