sqoop-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Fero Szabo via Review Board <nore...@reviews.apache.org>
Subject Re: Review Request 67524: SQOOP-3333 Change default behavior of the MS SQL connector to non-resilient.
Date Mon, 18 Jun 2018 09:26:55 GMT

This is an automatically generated e-mail. To reply, visit:

(Updated June 18, 2018, 9:26 a.m.)

Review request for Sqoop, Boglarka Egyed and Szabolcs Vasas.

Bugs: SQOOP-3333

Repository: sqoop-trunk

Description (updated)

This change is about changing the default behavior of the MS SQL connectore from resilient
to non-resilient. I was aiming for the fewest possible modifications while also removed double

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

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.

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/SqlServerManagerFormatConfigurator.java PRE-CREATION 
  src/test/org/apache/sqoop/manager/sqlserver/SQLServerManagerImportTest.java c83c2c93 
  src/test/org/apache/sqoop/manager/sqlserver/TestSqlServerManagerConfigurator.java PRE-CREATION

Diff: https://reviews.apache.org/r/67524/diff/2/

Changes: https://reviews.apache.org/r/67524/diff/1-2/


unit and 3rd party tests.
ant docs ran succesfully.
manual testing.


Fero Szabo

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