> On July 10, 2018, 4:05 p.m., Boglarka Egyed wrote:
> > .gitignore
> > Lines 39 (patched)
> > <https://reviews.apache.org/r/67424/diff/1/?file=2034783#file2034783line39>
> >
> >     Just as in https://reviews.apache.org/r/54528/ why do we need this here?

It happened to me a few times that "out" directory was created after running 
"ant jar-all" or "ant test" commands, as those are the ones I run most often. 
However, not every time though, so I am not sure what exactly will create this 
directory, but definitely happened to me several times. (I tested again, it was 
not created anymore)


> On July 10, 2018, 4:05 p.m., Boglarka Egyed wrote:
> > src/java/org/apache/sqoop/util/AppendUtils.java
> > Line 62 (original)
> > <https://reviews.apache.org/r/67424/diff/1/?file=2034784#file2034784line62>
> >
> >     Seems to be an unnecessary line modification.

Possibly during testing, reverted back.


> On July 10, 2018, 4:05 p.m., Boglarka Egyed wrote:
> > src/java/org/apache/sqoop/util/AppendUtils.java
> > Line 174 (original)
> > <https://reviews.apache.org/r/67424/diff/1/?file=2034784#file2034784line174>
> >
> >     Seems to be an unnecessary line modification.

Possibly during testing, reverted back.


- Eric


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


On June 2, 2018, 6:31 a.m., Eric Lin wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/67424/
> -----------------------------------------------------------
> 
> (Updated June 2, 2018, 6:31 a.m.)
> 
> 
> Review request for Sqoop and Boglarka Egyed.
> 
> 
> Bugs: SQOOP-3330
>     https://issues.apache.org/jira/browse/SQOOP-3330
> 
> 
> Repository: sqoop-trunk
> 
> 
> Description
> -------
> 
> When adding --append to Sqoop directory import with 
> -Dmapreduce.output.basename, all files will be ignored, which end up nothing 
> being imported. See below DEBUG output:
> 
> {code}
> sqoop import -Dmapreduce.output.basename="eric-test" --connect 
> jdbc:mysql://mysql-host.com/test --username root --password 'root' --table 
> test --target-dir /tmp/ericlin-test/sqoop/test --fields-terminated-by '\t' 
> --verbose --append
> 
> 18/05/28 22:24:44 INFO util.AppendUtils: Appending to directory test
> 18/05/28 22:24:44 DEBUG util.AppendUtils: Filename: _SUCCESS ignored
> 18/05/28 22:24:44 DEBUG util.AppendUtils: Filename: eric-test-m-00000 ignored
> 18/05/28 22:24:44 DEBUG util.AppendUtils: Filename: eric-test-m-00001 ignored
> 18/05/28 22:24:44 DEBUG util.AppendUtils: Filename: eric-test-m-00002 ignored
> 18/05/28 22:24:44 DEBUG util.AppendUtils: Deleting temporary folder 
> 14935e396acc4ea7b9a6236c66064c9b_test
> {code}
> 
> This is due to AppendUtils only recognizes file name starts with 
> "part.*-([0-9]"
> 
> https://github.com/apache/sqoop/blob/branch-1.4.7/src/java/org/apache/sqoop/util/AppendUtils.java#L46
> 
> 
> Diffs
> -----
> 
>   .gitignore 68cbe287 
>   src/java/org/apache/sqoop/util/AppendUtils.java fa852804 
>   src/test/org/apache/sqoop/TestAppendUtils.java f14fc6ad 
> 
> 
> Diff: https://reviews.apache.org/r/67424/diff/1/
> 
> 
> Testing
> -------
> 
> Added test case + manual test in VM
> 
> 
> Thanks,
> 
> Eric Lin
> 
>

Reply via email to