Home | About | Sematext search-lucene.com search-hadoop.com
 Search Hadoop and all its subprojects:

Switch to Plain View
Sqoop >> mail # dev >> Review Request: Review request for SQOOP-1033 "CombineFileInputFormat does not work with paths not on default FS like ASV"


+
Shuaishuai Nie 2013-05-08, 00:38
+
Venkat Ranganathan 2013-05-08, 16:00
+
Shuaishuai Nie 2013-05-08, 17:10
+
Shuaishuai Nie 2013-05-08, 17:11
+
Shuaishuai Nie 2013-05-08, 20:12
+
Shuaishuai Nie 2013-05-09, 22:16
+
Venkat Ranganathan 2013-05-10, 04:49
+
Shuaishuai Nie 2013-05-10, 14:51
+
Jarek Cecho 2013-05-20, 14:27
+
Venkat Ranganathan 2013-05-09, 21:57
+
Shuaishuai Nie 2013-05-08, 00:30
+
Venkat Ranganathan 2013-05-12, 15:48
+
Shuaishuai Nie 2013-05-13, 20:54
+
Shuaishuai Nie 2013-05-14, 00:31
+
Shuaishuai Nie 2013-05-16, 18:48
+
Venkat Ranganathan 2013-05-21, 20:03
+
Shuaishuai Nie 2013-05-23, 17:55
+
Venkat Ranganathan 2013-05-25, 00:37
+
Abhijeet Gaikwad 2013-06-03, 10:36
+
Jarek Cecho 2013-06-09, 16:02
+
Shuaishuai Nie 2013-06-12, 17:30
+
Jarek Cecho 2013-06-09, 16:23
Copy link to this message
-
Re: Review Request: Review request for SQOOP-1035 "Add MS Sqoop Connecter tests to repo"


> On June 9, 2013, 4:23 p.m., Jarek Cecho wrote:
> > src/test/org/apache/sqoop/manager/MSSQLTestUtils.java, lines 46-63
> > <https://reviews.apache.org/r/10987/diff/5/?file=295735#file295735line46>
> >
> >     Is there a linux equivalent for these commands?

This is no longer necessary for the integration test, so removed from the code
> On June 9, 2013, 4:23 p.m., Jarek Cecho wrote:
> > conf/MSTest.properties, lines 1-5
> > <https://reviews.apache.org/r/10987/diff/5/?file=295730#file295730line1>
> >
> >     The top level conf/ directory should be used only for code that is mean to be used for the actual product, not for the tests. Would it be feasible to move it somewhere else? For example to testdata/ or src/test?
> >    
> >     Also this file is missing Apache License.

moved to SQOOP_HOME/testdata/
> On June 9, 2013, 4:23 p.m., Jarek Cecho wrote:
> > testdata/DatatypeTestData-export-lite.txt, line 1
> > <https://reviews.apache.org/r/10987/diff/5/?file=295749#file295749line1>
> >
> >     This file do not have a license. We need to either put a license here or configure RAT to skip checking this file.

It seems the /testdata/ folder is excluded by rat in /src/scripts/rat-violation.sh already
> On June 9, 2013, 4:23 p.m., Jarek Cecho wrote:
> > testdata/DatatypeTestData-import-lite.txt, line 1
> > <https://reviews.apache.org/r/10987/diff/5/?file=295750#file295750line1>
> >
> >     This file do not have a license. We need to either put a license here or configure RAT to skip checking this file.

It seems the /testdata/ folder is excluded by rat in /src/scripts/rat-violation.sh already
- Shuaishuai
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/10987/#review21151
-----------------------------------------------------------
On May 23, 2013, 5:55 p.m., Shuaishuai Nie wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/10987/
> -----------------------------------------------------------
>
> (Updated May 23, 2013, 5:55 p.m.)
>
>
> Review request for Sqoop.
>
>
> Description
> -------
>
> Add the MS Sqoop connector tests that test integration scenarios with SQL Server to the repo.
>
>
> This addresses bug SQOOP-1035.
>     https://issues.apache.org/jira/browse/SQOOP-1035
>
>
> Diffs
> -----
>
>   conf/MSTest.properties PRE-CREATION
>   src/test/com/cloudera/sqoop/hive/TestHiveImport.java 462ccf1
>   src/test/com/cloudera/sqoop/manager/SQLServerManagerImportManualTest.java 27860c2
>   src/test/org/apache/sqoop/manager/MSSQLTestData.java PRE-CREATION
>   src/test/org/apache/sqoop/manager/MSSQLTestDataFileParser.java PRE-CREATION
>   src/test/org/apache/sqoop/manager/MSSQLTestUtils.java PRE-CREATION
>   src/test/org/apache/sqoop/manager/ManagerCompatExport.java PRE-CREATION
>   src/test/org/apache/sqoop/manager/SQLServerDatatypeExportDelimitedFileManualTest.java PRE-CREATION
>   src/test/org/apache/sqoop/manager/SQLServerDatatypeExportSequenceFileManualTest.java PRE-CREATION
>   src/test/org/apache/sqoop/manager/SQLServerDatatypeImportDelimitedFileManualTest.java PRE-CREATION
>   src/test/org/apache/sqoop/manager/SQLServerDatatypeImportSequenceFileManualTest.java PRE-CREATION
>   src/test/org/apache/sqoop/manager/SQLServerHiveImportManualTest.java PRE-CREATION
>   src/test/org/apache/sqoop/manager/SQLServerManagerManualTest.java PRE-CREATION
>   src/test/org/apache/sqoop/manager/SQLServerMultiColsManualTest.java PRE-CREATION
>   src/test/org/apache/sqoop/manager/SQLServerMultiMapsManualTest.java PRE-CREATION
>   src/test/org/apache/sqoop/manager/SQLServerParseMethodsManualTest.java PRE-CREATION
>   src/test/org/apache/sqoop/manager/SQLServerQueryManualTest.java PRE-CREATION
>   src/test/org/apache/sqoop/manager/SQLServerSplitByManualTest.java PRE-CREATION
>   src/test/org/apache/sqoop/manager/SQLServerWhereManualTest.java PRE-CREATION
+
Shuaishuai Nie 2013-06-12, 17:30
+
Venkat Ranganathan 2013-06-12, 17:51
+
Jarek Cecho 2013-06-15, 17:41
+
Shuaishuai Nie 2013-06-18, 20:57
+
Shuaishuai Nie 2013-06-18, 20:57
+
Jarek Cecho 2013-06-18, 22:09
+
Shuaishuai Nie 2013-06-18, 22:24
+
Shuaishuai Nie 2013-05-21, 21:51
+
Shuaishuai Nie 2013-05-21, 21:10
+
Shuaishuai Nie 2013-05-22, 16:59
+
Shuaishuai Nie 2013-05-30, 17:11
+
Venkat Ranganathan 2013-05-30, 17:49
+
Venkat Ranganathan 2013-05-30, 06:55
+
Venkat Ranganathan 2013-05-22, 06:48
+
Raghav Gautam 2013-05-31, 20:37
+
Jarek Cecho 2013-06-02, 14:07
+
Raghav Gautam 2013-06-06, 03:56
+
Raghav Gautam 2013-06-06, 03:56
+
Jarek Cecho 2013-06-09, 15:17
+
Raghav Gautam 2013-06-10, 21:29
+
Venkat Ranganathan 2013-06-10, 23:32
+
Jarek Cecho 2013-06-11, 18:54