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

Jason Lowe commented on TEZ-3962:
---------------------------------

Thanks for updating the patch!  The unit test failures are unrelated and are 
test timeouts caused by file sync changes in upstream Hadoop local dir 
allocator (which I believe they are undoing do to all the problems it caused).

+1 lgtm.  Committing this.


> Configuration decode leaks an Inflater object
> ---------------------------------------------
>
>                 Key: TEZ-3962
>                 URL: https://issues.apache.org/jira/browse/TEZ-3962
>             Project: Apache Tez
>          Issue Type: Bug
>    Affects Versions: 0.9.2, 0.10.0
>            Reporter: Gopal V
>            Assignee: Eric Wohlstadter
>            Priority: Major
>         Attachments: TEZ-3962.1.patch, TEZ-3962.2.patch
>
>
> {code}
> public static Configuration createConfFromByteString(ByteString byteString) 
> throws IOException {
> ...
>     InflaterInputStream uncompressIs = new 
> InflaterInputStream(byteString.newInput());
>     DAGProtos.ConfigurationProto confProto = 
> DAGProtos.ConfigurationProto.parseFrom(uncompressIs);
> {code}
> InflaterInputStream is never closed, this will get eventually collected - but 
> the off-heap buffers for Inflater leaks temporarily.



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)

Reply via email to