This is an automated email from the ASF dual-hosted git repository.

SteNicholas pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/celeborn.git

commit cf8d472718a597fa254bc4db5c38ec65abbdeaaf
Author: Cheng Pan <[email protected]>
AuthorDate: Tue May 26 20:17:38 2026 +0800

    [CELEBORN-2338] Remove hardcoded CELEBORN_PRINT_LAUNCH_COMMAND=0
    
    ### What changes were proposed in this pull request?
    
    Remove hardcoded `CELEBORN_PRINT_LAUNCH_COMMAND=0` in `bin/celeborn-class`
    
    ### Why are the changes needed?
    
    It should be picked from the environment variable.
    
    ### Does this PR resolve a correctness bug?
    
    - [ ] Yes
    
    ### Does this PR introduce _any_ user-facing change?
    
    - [ ] Yes
    
    ### How was this patch tested?
    
    ```
    $ CELEBORN_PRINT_LAUNCH_COMMAND=1 sbin/celeborn-cli --version
    ...
    Start to launch /opt/java/openjdk/bin/java -XX:+IgnoreUnrecognizedVMOptions 
-cp /opt/celeborn/conf::/opt/celeborn/cli-jars/*: 
org.apache.celeborn.cli.CelebornCli --version
    ...
    ```
    
    Closes #3702 from pan3793/CELEBORN-2338.
    
    Authored-by: Cheng Pan <[email protected]>
    Signed-off-by: SteNicholas <[email protected]>
---
 bin/celeborn-class | 3 ---
 1 file changed, 3 deletions(-)

diff --git a/bin/celeborn-class b/bin/celeborn-class
index 0ee4b213b..ad1f9087e 100755
--- a/bin/celeborn-class
+++ b/bin/celeborn-class
@@ -22,9 +22,6 @@ fi
 
 export CELEBORN_CONF_DIR="${CELEBORN_CONF_DIR:-"${CELEBORN_HOME}/conf"}"
 
-# print launch command for debug
-export CELEBORN_PRINT_LAUNCH_COMMAND="0"
-
 if [ -z "$CELEBORN_ENV_LOADED" ]; then
   export CELEBORN_ENV_LOADED=1
 

Reply via email to