sqoop-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Boglarka Egyed <b...@apache.org>
Subject Re: Review Request 63576: SQOOP-3243: Importing BLOB data causes "Stream closed" error on encrypted HDFS
Date Mon, 06 Nov 2017 12:27:32 GMT

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

(Updated Nov. 6, 2017, 12:27 p.m.)


Review request for Sqoop, Ferenc Szabo, Anna Szonyi, and Szabolcs Vasas.


Summary (updated)
-----------------

SQOOP-3243: Importing BLOB data causes "Stream closed" error on encrypted HDFS


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


Repository: sqoop-trunk


Description
-------

Importing BLOB data causes "Stream closed" error on encrypted HDFS because Sqoop tries to
close CryptoOutputStream twice in LobFile class. This error was ignored with Java 7 implementation
of FilterOutputStream butis propagated to upper levels in Java 8 thus the error occurs with
BLOB import.


Diffs
-----

  ivy.xml 601aa015c5052da1a287baec07cea4c26e143594 
  ivy/libraries.properties 2ef04f4fdeca4b6d82d306870cc715fbb98e56d4 
  src/java/org/apache/sqoop/io/LobFile.java 4e1bf2cb4a356ba1a2fc3aeca12859bbeb329cd4 
  src/test/com/cloudera/sqoop/io/TestLobFile.java 029758c531b0214b9b7f82de59d8fbae61bfdb0a

  src/test/org/apache/sqoop/hbase/TestHBasePutProcessor.java 73b31772d73594536f93b71ad9639c4d1a3e34e5

  src/test/org/apache/sqoop/mapreduce/TestJdbcExportJob.java 250ffa640b63fe06635a93895e2fe3c8fedfbbaa

  src/test/org/apache/sqoop/tool/TestImportTool.java 6335fb1aa3ff63c12ae82cb69e3f77ade3c48c53



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


Testing (updated)
-------

I have added a new test case, I also needed to bump the Mockito version and upon that some
other test cases needed to be updated as well due to some Mockito backward incompatibility
issues.
Ran unit and 3rd party tests successfully.


Thanks,

Boglarka Egyed


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