This is an automated email from the ASF dual-hosted git repository.
ethanfeng pushed a commit to branch branch-0.4
in repository https://gitbox.apache.org/repos/asf/incubator-celeborn.git
The following commit(s) were added to refs/heads/branch-0.4 by this push:
new e5c8b41d6 [CELEBORN-1197] Avoid using the sleep command with the s
suffix in bash scripts
e5c8b41d6 is described below
commit e5c8b41d6d32a597ccfd1fd74cbf3095c9f742a7
Author: sychen <[email protected]>
AuthorDate: Mon Dec 25 16:57:20 2023 +0800
[CELEBORN-1197] Avoid using the sleep command with the s suffix in bash
scripts
### What changes were proposed in this pull request?
Remove the s suffix of sleep
### Why are the changes needed?
MacOS
```bash
./sbin/start-all.sh
```
```
usage: sleep seconds
```
### Does this PR introduce _any_ user-facing change?
### How was this patch tested?
Closes #2187 from cxzl25/CELEBORN-1197.
Authored-by: sychen <[email protected]>
Signed-off-by: mingji <[email protected]>
(cherry picked from commit 2bf7062db3705d87fd51a8ccee2daa4b588b3dd5)
Signed-off-by: mingji <[email protected]>
---
sbin/start-all.sh | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/sbin/start-all.sh b/sbin/start-all.sh
index 359b5ea02..470e1c078 100755
--- a/sbin/start-all.sh
+++ b/sbin/start-all.sh
@@ -48,7 +48,7 @@ do
fi
done
# pause 5 seconds to make sure that master is ready.
-sleep 5s
+sleep 5
# start workers
for host in `echo "$HOST_LIST" | sed "s/#.*$//;/^$/d" | grep '\[worker\]' |
awk '{print $NF}'`
