commons-issues mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Tim Allison (JIRA)" <j...@apache.org>
Subject [jira] [Commented] (COMPRESS-385) Add detect() to CompressorStreamFactory
Date Fri, 14 Apr 2017 15:26:41 GMT

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

Tim Allison commented on COMPRESS-385:
--------------------------------------

I couldn't get a clean build locally, but all looks good based on coveralls?
{noformat}
Failed tests: 
  ArchiveReadTest.data:77
Tests in error: 
  LongPathTest.data:85 NullPointer
  LongSymLinkTest.data:85 NullPointer
{noformat}

> Add detect() to CompressorStreamFactory
> ---------------------------------------
>
>                 Key: COMPRESS-385
>                 URL: https://issues.apache.org/jira/browse/COMPRESS-385
>             Project: Commons Compress
>          Issue Type: Improvement
>            Reporter: Tim Allison
>            Priority: Minor
>
> On TIKA-1631, several users have requested that we try to avoid an OOM when a corrupted
Z file is "detected" by CompressorStreamFactory.  
> In Tika, for detection, we're creating the stream via CompressorStreamFactory and then
"detecting" based on what stream was created.  Given that there can be some overhead in creating
the stream and that there can be an OOM for a corrupt Z file, it would be great to add a {{detect(InputStream
is)}} option in CompressorStreamFactory.
> PR on way.



--
This message was sent by Atlassian JIRA
(v6.3.15#6346)

Mime
View raw message