Review Request 67524: SQOOP-3333 Change default behavior of the MS SQL connector to non-resilient.

2018-06-11 Thread Fero Szabo via Review Board 

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

Review request for Sqoop, Boglarka Egyed and Szabolcs Vasas.


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


Repository: sqoop-trunk


Description
---

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 negation.

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.


Diffs
-

  src/docs/user/connectors.txt 7c540718 
  src/java/org/apache/sqoop/manager/SQLServerManager.java b136087f 


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


Testing
---

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


Thanks,

Fero Szabo

Re: Review Request 66300: Upgrade to Hadoop 3.0.0

2018-06-11 Thread Boglarka Egyed 

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



Hi Daniel,

Could you please rebase your patch to the latest trunk?

Thank you very much,
Bogi

- Boglarka Egyed


On March 27, 2018, 8:50 a.m., daniel voros wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/66300/
> ---
> 
> (Updated March 27, 2018, 8:50 a.m.)
> 
> 
> Review request for Sqoop.
> 
> 
> Bugs: SQOOP-3305
> https://issues.apache.org/jira/browse/SQOOP-3305
> 
> 
> Repository: sqoop-trunk
> 
> 
> Description
> ---
> 
> To be able to eventually support the latest versions of Hive, HBase and 
> Accumulo, we should start by upgrading our Hadoop dependencies to 3.0.0. See 
> https://hadoop.apache.org/docs/r3.0.0/index.html
> 
> 
> Diffs
> -
> 
>   ivy.xml 1f587f3e 
>   ivy/libraries.properties 565a8bf5 
>   src/java/org/apache/sqoop/SqoopOptions.java d9984af3 
>   src/java/org/apache/sqoop/config/ConfigurationHelper.java fb2ab031 
>   src/java/org/apache/sqoop/hive/HiveImport.java 5da00a74 
>   src/java/org/apache/sqoop/mapreduce/JobBase.java 6d1e0499 
>   src/java/org/apache/sqoop/mapreduce/ParquetJob.java 46047733 
>   src/java/org/apache/sqoop/mapreduce/hcat/DerbyPolicy.java PRE-CREATION 
>   src/java/org/apache/sqoop/mapreduce/hcat/SqoopHCatUtilities.java 784b5f2a 
>   src/java/org/apache/sqoop/util/SqoopJsonUtil.java adf186b7 
>   src/test/org/apache/sqoop/TestSqoopOptions.java bb7c20dd 
>   src/test/org/apache/sqoop/hive/minicluster/HiveMiniCluster.java 19bb7605 
>   
> src/test/org/apache/sqoop/hive/minicluster/KerberosAuthenticationConfiguration.java
>  549a8c6c 
>   
> src/test/org/apache/sqoop/hive/minicluster/PasswordAuthenticationConfiguration.java
>  79881f7b 
>   src/test/org/apache/sqoop/util/TestSqoopJsonUtil.java fdf972c1 
>   testdata/hcatalog/conf/hive-site.xml edac7aa9 
> 
> 
> Diff: https://reviews.apache.org/r/66300/diff/6/
> 
> 
> Testing
> ---
> 
> Normal and third-party unit tests.
> 
> 
> Thanks,
> 
> daniel voros
> 
>