----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56737/#review166651 -----------------------------------------------------------
Ship it! Ship It! - Attila Szabo On Feb. 23, 2017, 1:16 a.m., Eric Lin wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/56737/ > ----------------------------------------------------------- > > (Updated Feb. 23, 2017, 1:16 a.m.) > > > Review request for Sqoop and Attila Szabo. > > > Bugs: SQOOP-3135 > https://issues.apache.org/jira/browse/SQOOP-3135 > > > Repository: sqoop-trunk > > > Description > ------- > > Run the following sqoop command: > > sqoop job --create test -- import --connect jdbc:mysql://localhost/test > --username root --password $pass --target-dir /tmp/test10 -m 1 --driver > com.mysql.jdbc.Driver --table test > > Due to $pass is not set, command will fail with the following error: > > 16/12/21 05:48:32 INFO sqoop.Sqoop: Running Sqoop version: 1.4.7-SNAPSHOT > 16/12/21 05:48:33 ERROR tool.JobTool: Error parsing arguments to the > job-specific tool. > 16/12/21 05:48:33 ERROR tool.JobTool: See 'sqoop help <tool>' for usage. > > This is not informative, by checking the code in JobTool class: > > // Now feed the arguments into the tool itself. > try { > childOptions = childTool.parseArguments(parseableChildArgv, > null, childOptions, false); > childTool.appendArgs(extraChildArgv); > childTool.validateOptions(childOptions); > } catch (ParseException pe) { > LOG.error("Error parsing arguments to the job-specific tool."); > LOG.error("See 'sqoop help <tool>' for usage."); > return 1; > } catch (SqoopOptions.InvalidOptionsException e) { > System.err.println(e.getMessage()); > return 1; > } > > The ParseException pe's message has been dropped, we should print out the > message in the exception so that more meaningful message will be printed. > > > Diffs > ----- > > src/java/org/apache/sqoop/tool/JobTool.java 8e1669c > > Diff: https://reviews.apache.org/r/56737/diff/ > > > Testing > ------- > > Manual test and confirmed the output. > > > Thanks, > > Eric Lin > >
