sqoop-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "James Anderson" <anderson....@gmail.com>
Subject Re: Review Request 14542: Review of SQOOP-1213
Date Wed, 09 Oct 2013 14:57:16 GMT

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

(Updated Oct. 9, 2013, 2:57 p.m.)


Review request for Sqoop.


Changes
-------

Thanks for the feedback, Jarcec.  I've switched to using Path.getFileSystem(Configuration)
as suggested.  An updated version of the patch is attached to the JIRA ticket and this review
request.


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


Repository: sqoop-trunk


Description
-------

In order to support reading password files from Amazon S3, instantiate the FileSystem object
with the appropriate URI based on the provided path.


Diffs (updated)
-----

  src/java/org/apache/sqoop/util/CredentialsUtil.java 2d88bd3 

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


Testing
-------


Thanks,

James Anderson


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