GeoServer
  1. GeoServer
  2. GEOS-1644

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

        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
        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
        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
        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
        Justin Deoliveira added a comment -
        Show
        Justin Deoliveira added a comment - Integrated in geoserver-trunk #117 (See http://gridlock.openplans.org:8080/hudson/job/geoserver-trunk/117/ )
        Hide
        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
        Justin Deoliveira added a comment -
        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
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved: