-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/67524/
-----------------------------------------------------------
(Updated June 18, 2018, 2:48 p.m.)
Review request for Sqoop, Boglarka Egyed and Szabolcs Vasas.
Bugs: SQOOP-3333
https://issues.apache.org/jira/browse/SQOOP-3333
Repository: sqoop-trunk
Description
-------
This change is about changing the default behavior of the MS SQL connector from
resilient to non-resilient. I was aiming for the fewest possible modifications
while also removed double negation where previously present.
I've refactored the context configuration into a separate class.
I've also changed the documentation of the non-resilient flag and added a note
about the implicit requirement of the feature (that the split-by column has to
be unique and ordered in ascending order).
I plan to expand the documentation more in SQOOP-3332, as the (now named)
resilient flag works not just for export, but import as well (queries and
tables).
I've also added new tests that cover what classes get loaded in connection with
the resilient option. Also, I've refactored SQL Server import tests and added a
few more cases for better coverage. (The query import uses a different method
and wasn't covered by these tests at all.)
Diffs (updated)
-----
src/docs/user/connectors.txt 7c540718
src/java/org/apache/sqoop/manager/ExportJobContext.java 773cf742
src/java/org/apache/sqoop/manager/SQLServerManager.java b136087f
src/java/org/apache/sqoop/manager/SqlServerManagerContextConfigurator.java
PRE-CREATION
src/test/org/apache/sqoop/manager/sqlserver/SQLServerManagerImportTest.java
c83c2c93
src/test/org/apache/sqoop/manager/sqlserver/TestSqlServerManagerContextConfigurator.java
PRE-CREATION
Diff: https://reviews.apache.org/r/67524/diff/4/
Changes: https://reviews.apache.org/r/67524/diff/3-4/
Testing
-------
Added new unit tests for SqlServerConfigurator.
unit and 3rd party tests.
ant docs ran succesfully.
manual testing.
Thanks,
Fero Szabo
