small changes

Author: rileythomp

_run-druid.sh

@@ -7,6 +7,7 @@ source "$SOURCE_DIR/env-druid.sh"
 source "$SOURCE_DIR/common.sh"
 
 kill-existing ${CONTAINER_NAME}
+
 docker run \
        -p ${WEB_PORT}:8081 \
        -p ${HOST_PORT}:8082 \

_run-postgres.sh

@@ -6,35 +6,35 @@ source "$SOURCE_DIR/env-postgres.sh"
 
 source "$SOURCE_DIR/common.sh"
 
-DATA_DIR=$HOME/metabase-pgsql-${PG_VERSION}-data
+DATA_DIR=${HOME}/metabase-pgsql-${PG_VERSION}-data
 DOCKER_NETWORK=psql-metabase-network
 
 kill-existing ${CONTAINER_NAME}
-create-network-if-needed $DOCKER_NETWORK
+create-network-if-needed ${DOCKER_NETWORK}
 
 docker run \
        --rm \
        -d \
-       -p $HOST_PORT:5432 \
-       --network $DOCKER_NETWORK \
-       -e POSTGRES_USER=$DB_USER \
-       -e POSTGRES_DB=$DB_NAME \
-       -e POSTGRES_PASSWORD=$DB_PASSWORD \
+       -p ${HOST_PORT}:5432 \
+       --network ${DOCKER_NETWORK} \
+       -e POSTGRES_USER=${DB_USER} \
+       -e POSTGRES_DB=${DB_NAME} \
+       -e POSTGRES_PASSWORD=${DB_PASSWORD} \
        -e PGDATA=/var/lib/postgresql/data \
-       -v $DATA_DIR:/var/lib/postgresql/data:Z \
-       --name $CONTAINER_NAME \
+       -v ${DATA_DIR}:/var/lib/postgresql/data:Z \
+       --name ${CONTAINER_NAME} \
        postgres:${PG_VERSION}
 
 cat << EOF
 
-Started PostgreSQL ${PG_VERSION} on port $HOST_PORT via Docker (container name: $CONTAINER_NAME).
-Data will be persisted in $DATA_DIR on the host machine (delete it to reset).
+Started PostgreSQL ${PG_VERSION} on port ${HOST_PORT} via Docker (container name: ${CONTAINER_NAME}).
+Data will be persisted in ${DATA_DIR} on the host machine (delete it to reset).
 
 To open a SQL client session:
-docker run -it --rm --network $DOCKER_NETWORK postgres:${PG_VERSION} psql -h $CONTAINER_NAME -U $DB_USER
-And enter the DB user password for $DB_USER: $DB_PASSWORD
+docker run -it --rm --network ${DOCKER_NETWORK} postgres:${PG_VERSION} psql -h ${CONTAINER_NAME} -U ${DB_USER}
+And enter the DB user password for ${DB_USER}: ${DB_PASSWORD}
 
-JDBC URL: jdbc:postgres://localhost:$HOST_PORT/$DB_NAME?user=$DB_USER
+JDBC URL: jdbc:postgres://localhost:${HOST_PORT}/${DB_NAME}?user=${DB_USER}
 
 EOF
 

_run-presto.sh

@@ -43,7 +43,7 @@ also have the JAVA_HOME env var set for this all to work.
 Finally, this can be made available to Metabase by adding the following JVM args when starting:
 
   -Djavax.net.ssl.trustStore=$MODIFIED_CACERTS_FILE -Djavax.net.ssl.trustStorePassword=changeit
-  
+
 EOF
 
 print-presto-vars

_run-sparksql.sh

@@ -1,7 +1,5 @@
 #! /usr/bin/env bash
 
-set -euo pipefail
-
 SOURCE_DIR=`dirname "${BASH_SOURCE[0]}"`
 
 source "$SOURCE_DIR/env-sparksql.sh"

_run-vertica.sh

@@ -24,7 +24,7 @@ cat <<EOF
 
 Started Vertica ${VERTICA_VERSION} on port ${HOST_PORT}.
 
-JDBC URL: "jdbc:vertica://localhost:${HOST_PORT}/${DB_NAME}?user=${DB_USER}"
+JDBC URL: jdbc:vertica://localhost:${HOST_PORT}/${DB_NAME}?user=${DB_USER}
 
 EOF