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
The following commit(s) were added to refs/heads/main by this push:
new dc3c421ad [CELEBORN-2338] Remove hardcoded
CELEBORN_PRINT_LAUNCH_COMMAND=0
dc3c421ad is described below
commit dc3c421ad0a374c3732bf38081a59557e5ad98ab
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: 子懿 <[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
