You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@arrow.apache.org by ag...@apache.org on 2022/10/26 13:12:19 UTC

[arrow-ballista] branch master updated: Add set -e to scripts (#444)

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

agrove pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/arrow-ballista.git


The following commit(s) were added to refs/heads/master by this push:
     new ef43d36c Add set -e to scripts (#444)
ef43d36c is described below

commit ef43d36c53dbf62db26ddeadcf48938eafedf099
Author: Andy Grove <an...@gmail.com>
AuthorDate: Wed Oct 26 07:12:13 2022 -0600

    Add set -e to scripts (#444)
---
 dev/docker/builder-entrypoint.sh   | 3 ++-
 dev/docker/executor-entrypoint.sh  | 2 ++
 dev/docker/scheduler-entrypoint.sh | 2 ++
 3 files changed, 6 insertions(+), 1 deletion(-)

diff --git a/dev/docker/builder-entrypoint.sh b/dev/docker/builder-entrypoint.sh
index f103b5e7..52137d59 100755
--- a/dev/docker/builder-entrypoint.sh
+++ b/dev/docker/builder-entrypoint.sh
@@ -17,10 +17,11 @@
 # specific language governing permissions and limitations
 # under the License.
 
+set -e
 set -x
 
 printenv
-RELEASE_FLAG=${RELEASE_FLA:=release}
+RELEASE_FLAG=${RELEASE_FLAG:=release}
 cargo build --features flight-sql "--$RELEASE_FLAG" "$@"
 
 cd ballista/scheduler/ui
diff --git a/dev/docker/executor-entrypoint.sh b/dev/docker/executor-entrypoint.sh
index 5f749a2a..a1df6d58 100755
--- a/dev/docker/executor-entrypoint.sh
+++ b/dev/docker/executor-entrypoint.sh
@@ -17,4 +17,6 @@
 # specific language governing permissions and limitations
 # under the License.
 
+set -e
+
 /root/ballista-executor "$@"
diff --git a/dev/docker/scheduler-entrypoint.sh b/dev/docker/scheduler-entrypoint.sh
index 2f5a1d0b..aab8140a 100755
--- a/dev/docker/scheduler-entrypoint.sh
+++ b/dev/docker/scheduler-entrypoint.sh
@@ -17,6 +17,8 @@
 # specific language governing permissions and limitations
 # under the License.
 
+set -e
+
 echo "Starting nginx to serve Ballista Scheduler web UI on port 80"
 nohup nginx -g "daemon off;" &
 /root/ballista-scheduler "$@"