linghengqian commented on code in PR #39146:
URL: https://github.com/apache/shardingsphere/pull/39146#discussion_r3594405025


##########
distribution/proxy-native/Dockerfile-linux-dynamic:
##########
@@ -25,4 +25,4 @@ LABEL org.opencontainers.image.authors="ShardingSphere 
[email protected]
 ENV LOCAL_PATH=/opt/shardingsphere-proxy
 ARG PROJECT_VERSION
 COPY --from=nativebuild 
/build/distribution/proxy-native/target/apache-shardingsphere-${PROJECT_VERSION}-shardingsphere-proxy-bin
 ${LOCAL_PATH}
-ENTRYPOINT ["${LOCAL_PATH}/bin/shardingsphere-proxy-native", "3307", 
"${LOCAL_PATH}/conf", "0.0.0.0"]
+ENTRYPOINT ["/opt/shardingsphere-proxy/bin/shardingsphere-proxy-native", 
"3307", "/opt/shardingsphere-proxy/conf", "0.0.0.0"]

Review Comment:
   I’ve somewhat forgotten the initial details; after all, for a long time, 
GraalVM Native Image could only be debugged using GDB, and support for 
debugging it via JDWP has only arrived within the last year.
   
   So, why didn't you go ahead and remove the `ENV 
LOCAL_PATH=/opt/shardingsphere-proxy` definition as well?



-- 
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: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to