hive-issues mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Ted Yu (JIRA)" <j...@apache.org>
Subject [jira] [Commented] (HIVE-7862) close of InputStream in Utils#copyToZipStream() should be placed in finally block
Date Tue, 08 Mar 2016 22:11:40 GMT

    [ https://issues.apache.org/jira/browse/HIVE-7862?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15185933#comment-15185933
] 

Ted Yu commented on HIVE-7862:
------------------------------

Test failures didn't seem to be related.

> close of InputStream in Utils#copyToZipStream() should be placed in finally block
> ---------------------------------------------------------------------------------
>
>                 Key: HIVE-7862
>                 URL: https://issues.apache.org/jira/browse/HIVE-7862
>             Project: Hive
>          Issue Type: Bug
>    Affects Versions: 0.13.0
>            Reporter: Ted Yu
>            Assignee: skrho
>            Priority: Minor
>              Labels: patch
>         Attachments: HIVE-7862.1.patch, HIVE-7862.1.patch, HIVE-7862_001.txt
>
>
> In accumulo-handler/src/java/org/apache/hadoop/hive/accumulo/Utils.java , line 278 :
> {code}
>   private static void copyToZipStream(InputStream is, ZipEntry entry, ZipOutputStream
zos)
>       throws IOException {
>     zos.putNextEntry(entry);
>     byte[] arr = new byte[4096];
>     int read = is.read(arr);
>     while (read > -1) {
>       zos.write(arr, 0, read);
>       read = is.read(arr);
>     }
>     is.close();
> {code}
> If read() throws IOException, is would be left unclosed.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

Mime
View raw message