You are viewing a plain text version of this content. The canonical link for it is here.

Posted to commits@inlong.apache.org by GitBox <gi...@apache.org> on 2022/07/19 09:28:05 UTC

[GitHub] [inlong] lucaspeng12138 commented on a diff in pull request #5125: [INLONG-5124][Docker] Add initial sort connectors to the manager docker image

lucaspeng12138 commented on code in PR #5125:
URL: https://github.com/apache/inlong/pull/5125#discussion_r924272894


##########
inlong-manager/manager-docker/Dockerfile:
##########
@@ -37,10 +37,14 @@ ENV AUDIT_PROXY_URL=127.0.0.1:10081
 # JVM env
 ENV MANAGER_JVM_HEAP_OPTS="-XX:+UseContainerSupport -XX:InitialRAMPercentage=40.0 -XX:MaxRAMPercentage=80.0 -XX:-UseAdaptiveSizePolicy"
 WORKDIR /opt/inlong-manager
-
-# add tarball from manager output
-ARG MANAGER_TARBALL

Review Comment:
   docker/build-docker-images.sh has used MANAGER_TARBALL at line 122, need to change into version



-- 
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: commits-unsubscribe@inlong.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org