Sorry to not notice this earlier,but given the default value, I think the equality check with "client" now may make more sense.

Also, along similar lines, perhaps we could use `-n` instead of `! -z` for DRIVER_MEMORY, and also wrap the variables in quotes. For instance, right now if $DRIVER_MEMORY is not set, this is actually evaluating `if [ ! -z ]`, which happens to evaluate to false, but is confusing because `if [ ! -n ]` also evaluates to false. By putting quotes around it, we'll make sure we actually evaluate `if [ -n "" ]` if it's empty, which has the semantics we're actually looking for.