-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/38589/#review100960
-----------------------------------------------------------

dist/src/main/bin/sqoop.sh (lines 29 - 33)
<https://reviews.apache.org/r/38589/#comment158248>

    Wouldn't it make more sense to make this check after we define the variables as we're setting up defaults there?
   
    Also rather then verifying that those variables are non empty, would it make sense to verify that those variables describes valid directories (e.g. -d instead of -n).
- Jarek Cecho
On Sept. 29, 2015, 8:18 a.m., Colin Ma wrote:

NEW: Monitor These Apps!
elasticsearch, apache solr, apache hbase, hadoop, redis, casssandra, amazon cloudwatch, mysql, memcached, apache kafka, apache zookeeper, apache storm, ubuntu, centOS, red hat, debian, puppet labs, java, senseiDB