GeoServer

Disable the clibpngimagereader by default

Details

  • Type: Improvement Improvement
  • Status: Closed Closed
  • Priority: Major Major
  • Resolution: Fixed
  • Affects Version/s: 1.6.0-RC2
  • Fix Version/s: 1.6.1
  • Component/s: WCS, WMS
  • Labels:
    None
  • Number of attachments :
    0

Description

Tests show the clib png image reader is slower than the pure java one and has higher memory requirements. We should disable it globally when GeoServer starts up, or have mosaic, pyramid and worldimage avoid its usage.

Activity

Ascending order - Click to sort in descending order
Hide
Permalink
Andrea Aime added a comment -

References to the tests and associated mails:
https://jai-imageio.dev.java.net/servlets/ReadMsg?list=interest&msgNo=2302
https://jai-imageio.dev.java.net/servlets/ReadMsg?list=interest&msgNo=2311

Show
Andrea Aime added a comment - References to the tests and associated mails: https://jai-imageio.dev.java.net/servlets/ReadMsg?list=interest&msgNo=2302 https://jai-imageio.dev.java.net/servlets/ReadMsg?list=interest&msgNo=2311
Hide
Permalink
Martin Desruisseaux added a comment -

We were aware that JAI native codecs have some issues at least for the PNG format, and an utility method was created a few years ago for forcing the pure Java codecs. In GeoTools 2.5:

ImageUtilities.allowNativeCodec("png", ImageReaderSpi.class, false);
ImageUtilities.allowNativeCodec("png", ImageWriterSpi.class, false);

(the method signature was sightly different in GeoTools 2.4 and before). The ImageUtilities class is in org.geotools.resources.image package if my memory serve me right.

Show
Martin Desruisseaux added a comment - We were aware that JAI native codecs have some issues at least for the PNG format, and an utility method was created a few years ago for forcing the pure Java codecs. In GeoTools 2.5: ImageUtilities.allowNativeCodec( "png" , ImageReaderSpi.class, false ); ImageUtilities.allowNativeCodec( "png" , ImageWriterSpi.class, false ); (the method signature was sightly different in GeoTools 2.4 and before). The ImageUtilities class is in org.geotools.resources.image package if my memory serve me right.
Hide
Permalink
Andrea Aime added a comment -

Ah ha, good to know, thanks Martin.

Show
Andrea Aime added a comment - Ah ha, good to know, thanks Martin.
Hide
Permalink
Andrea Aime added a comment -

Fix committed to trunk for the moment, waiting for 1.6.0 release to commit on 1.6.x too

Show
Andrea Aime added a comment - Fix committed to trunk for the moment, waiting for 1.6.0 release to commit on 1.6.x too
Hide
Permalink
Justin Deoliveira added a comment -

Integrated in geoserver-trunk #117 (See http://gridlock.openplans.org:8080/hudson/job/geoserver-trunk/117/)

Show
Justin Deoliveira added a comment - Integrated in geoserver-trunk #117 (See http://gridlock.openplans.org:8080/hudson/job/geoserver-trunk/117/ )
Hide
Permalink
Andrea Aime added a comment -

Fixed on 1.6.x as well

Show
Andrea Aime added a comment - Fixed on 1.6.x as well
Hide
Permalink
Justin Deoliveira added a comment -

Integrated in geoserver-1.6.x #46 (See http://gridlock.openplans.org:8080/hudson/job/geoserver-1.6.x/46/)

Show
Justin Deoliveira added a comment - Integrated in geoserver-1.6.x #46 (See http://gridlock.openplans.org:8080/hudson/job/geoserver-1.6.x/46/ )

People

  • Assignee:
    Andrea Aime
    Reporter:
    Andrea Aime
Vote (0)
Watch (0)

Dates

  • Created:
    Updated:
    Resolved: