sqoop-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Clément MATHIEU <clem...@unportant.info>
Subject Re: Review Request 38372: Support of glob paths during export
Date Sat, 03 Oct 2015 15:00:33 GMT

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

(Updated oct. 3, 2015, 3 après-midi)


Review request for Sqoop.


Changes
-------

Add an unit test


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


Repository: sqoop-trunk


Description
-------

The only thing required to add support of glob paths is to update ExportJobBase.getFileType.

Current code fails with an exception when globbing is used. The patch add support of glob
paths while preserving the original logic:

 - Returns the type of the first file
 - Fails if no file can be found


Diffs (updated)
-----

  src/java/org/apache/sqoop/mapreduce/ExportJobBase.java f9fa7f3 
  src/test/com/cloudera/sqoop/TestAvroExport.java 663828c 

Diff: https://reviews.apache.org/r/38372/diff/


Testing
-------

It has been tested & used with Sqoop 1.4.5 on top of a CDH 4.7, 5.3 & 5.4 (we have
jobs taking glob paths as input parameter)

I quickly crawled the test suite to see how I could unit test this change but failed to find
a trivial implementation. I believe that I have to fiddle with getTablePath / TestExport but
I'm not sure where it will take me. Any advice is welcome.


Thanks,

Clément MATHIEU


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