sqoop-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Seetharam Venkatesh" <venkat...@innerzeal.com>
Subject Review Request: Adds an abort validation handler which terminates the processing of import/export jobs.
Date Wed, 03 Apr 2013 10:53:47 GMT

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

Review request for Sqoop and Jarek Cecho.


Description
-------

Validation of the data copied, either import or export, is implemented today by comparing
the row counts from the source and the target post copy. This issue adds a handler that aborts
the processing notifying the user of the anomaly.

Makes this the default implementation.


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


Diffs
-----

  src/docs/user/validation.txt f28b420 
  src/java/org/apache/sqoop/SqoopOptions.java 906542b 
  src/java/org/apache/sqoop/validation/AbortOnFailureHandler.java PRE-CREATION 
  src/java/org/apache/sqoop/validation/LogOnFailureHandler.java 3ded6ad 
  src/java/org/apache/sqoop/validation/RowCountValidator.java 2896192 
  src/java/org/apache/sqoop/validation/ValidationException.java 0c10241 
  src/test/org/apache/sqoop/validation/AbortOnFailureHandlerTest.java PRE-CREATION 
  src/test/org/apache/sqoop/validation/AbsoluteValidationThresholdTest.java PRE-CREATION 
  src/test/org/apache/sqoop/validation/RowCountValidatorImportTest.java a53e281 

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


Testing
-------

Yes. Unit tests are added to test the new functionality.


Thanks,

Seetharam Venkatesh


Mime
  • Unnamed multipart/alternative (inline, None, 0 bytes)
View raw message