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

Review request for Sqoop.


Bugs: SQOOP-3291
    https://issues.apache.org/jira/browse/SQOOP-3291


Repository: sqoop-trunk


Description
-------

Job data is published to listeners (defined via sqoop.job.data.publish.class) 
in case of Hive and HCat imports. Currently this happens before the Hive import 
completes, so it gets reported even if Hive import fails.


Diffs
-----

  src/java/org/apache/sqoop/hive/HiveImport.java c272911 
  src/java/org/apache/sqoop/mapreduce/ExportJobBase.java 6529bd2 
  src/java/org/apache/sqoop/mapreduce/ImportJobBase.java fb5d054 
  src/java/org/apache/sqoop/mapreduce/PublishJobData.java fc18188 
  src/java/org/apache/sqoop/tool/ImportTool.java e992005 
  src/test/org/apache/sqoop/TestSqoopJobDataPublisher.java b3579ac 
  src/test/org/apache/sqoop/testutil/BaseSqoopTestCase.java a5f85a0 


Diff: https://reviews.apache.org/r/65884/diff/1/


Testing
-------

- created unit test
 - tested on a cluster with Atlas SqoopHook in place


Thanks,

daniel voros

Reply via email to