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

Review request for Sqoop.


Description
-------

I've used following command to change all error exception logging to call 
method that properly resolves "getNextException":

grep "(SQLException" src/java -R -A 2 -n | grep "LOG.error"


This addresses bug SQOOP-933.
    https://issues.apache.org/jira/browse/SQOOP-933


Diffs
-----

  src/java/org/apache/sqoop/manager/CatalogQueryManager.java 
fa7661e38e5cf0fd9ad202d3953e7b64f8a4bf8c 
  src/java/org/apache/sqoop/manager/Db2Manager.java 
a84b4dcd2b7101f0eb7bb4ea30e104c7aad6d677 
  src/java/org/apache/sqoop/manager/MySQLManager.java 
b79c92664462e80bb6e1028cf14d569e805f0bbf 
  src/java/org/apache/sqoop/manager/OracleManager.java 
56244c741d9221b01cab27454bb18be4937ff639 
  src/java/org/apache/sqoop/manager/SqlManager.java 
f0a920df4d423ec0caa4efb34cd7678a8b8f54b9 
  src/java/org/apache/sqoop/mapreduce/ExportJobBase.java 
5261671bdc3461f57fa3171d12447aeba2079444 
  src/java/org/apache/sqoop/mapreduce/PGBulkloadExportMapper.java 
6c8d8068e73bc844b309f62726ec4fb9543abc67 
  src/java/org/apache/sqoop/mapreduce/PGBulkloadExportReducer.java 
63c52c7bedfdea6fb3d85642dc8f9cb78b1a1e07 
  src/java/org/apache/sqoop/mapreduce/db/DBConfiguration.java 
d270bc87dd793a0c7f30888571e8048a98a7f707 
  src/java/org/apache/sqoop/mapreduce/db/DBOutputFormat.java 
cd37a25e1a06dc34c8066370c53df0161796a093 
  src/java/org/apache/sqoop/mapreduce/db/DBRecordReader.java 
ab8bf1894149e50e4233113b112832ef87c0fe06 
  src/java/org/apache/sqoop/mapreduce/db/DataDrivenDBInputFormat.java 
81677d7133f91acec5961ca4d91dd8f1575b6395 
  src/java/org/apache/sqoop/util/LoggingUtils.java 
cb74401be65cf181fc0000e7cef45982b99eeae2 
  src/java/org/apache/sqoop/util/ResultSetPrinter.java 
c6ec09df70c8643e1b8b5d26e4248cefcacded59 

Diff: https://reviews.apache.org/r/9803/diff/


Testing
-------

Unit tests are passing.


Thanks,

Jarek Cecho

Reply via email to