commons-issues mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Stefan Bodewig (JIRA)" <>
Subject [jira] [Commented] (COMPRESS-285) checking of availability of XZ compression is expensive - result should be reused
Date Sat, 05 Jul 2014 14:22:34 GMT


Stefan Bodewig commented on COMPRESS-285:

TBH you probably know more about OSGi than me :-)

What is important to me:

* some people use Commons Compress in an OSGi context and we must be good citizens in that
* right now we only provide manifest attributes to support OSGi, in particular we haven't
got any code dependency on any OSGi specific classes.  I don't want to change that.

If there is some caching solution that doesn't violate either point I'll be happly to apply

> checking of availability of XZ compression is expensive - result should be reused
> ---------------------------------------------------------------------------------
>                 Key: COMPRESS-285
>                 URL:
>             Project: Commons Compress
>          Issue Type: Improvement
>          Components: Compressors
>    Affects Versions: 1.5, 1.6, 1.7, 1.8
>         Environment: linux 64-bit, java 7, glassfish, solr, tika
>            Reporter: Wojciech Ɓozowicki
>            Priority: Minor
>              Labels: performance
> I use solr with apache tika for indexing documents. Tika uses commons-compress to handle
compressed files. Using sampler (jvisualvm) I have seen that quite a lot of time (5-7%) during
my tests is spent in XZUtils.isXZCompressionAvailable because of unavailable XZ compression
(I guess for each time classloaders spend some time looking for unavailable classes, then
> I think the result of the first check should be stored and reused.
> Here is the stacktrace (just to show the way tika is using commons-compress):
> org.apache.commons.compress.compressors.xz.XZUtils.isXZCompressionAvailable(
> 	at org.apache.commons.compress.compressors.CompressorStreamFactory.createCompressorInputStream(
> 	at org.apache.tika.parser.pkg.ZipContainerDetector.detectCompressorFormat(
> 	at org.apache.tika.parser.pkg.ZipContainerDetector.detect(
> 	at org.apache.tika.detect.CompositeDetector.detect(

This message was sent by Atlassian JIRA

View raw message