That sounds fine; in that we should allow for expected behaviour.
How do you feel about removing .wrapGridCoverageReader? We only did this to
work around issues in the origional MapLayer (that was only willing to store
features).
If you update the renderer to use GridReaderLayer I think you will avoid this
problem completely? (GridReaderlayer uses wrapGridCoverageReader in the
toFeatureCollection method in order to preserve that status quo).
--
Jody Garnett
On Friday, 6 May 2011 at 11:26 PM, Gabriel Roldán wrote:
> Hi all,
>
> sorry for the cross-posting.
>
> I wonder what you think of explaining
> org.geotools.data.DataUtilities.defaultValue that
> AbstractGridCoverage2DReader and GeneralParameterValue are valid
> arguments to that function and hence it should return null instead of
> throwing IllegalArgumentException.
>
> Thing is that everytime geoserver renders a coverage two
> IllegalArgumentException are thrown per request, comming from:
>
> wms.GetMap.run->geotools.resources.coverage.FeatureUtilities.wrapGridCoverageReader->SimpleFeatureTypeBuilder.add(String,
>
> class)->AttributeTypeBuilder.setBinding(Class)->geotools.data.DataUtilities.defaultValue(Class),
> for arguments:
> * org.opengis.parameter.GeneralParameterValue
> * org.geotools.coverage.grid.io.AbstractGridCoverage2DReader
>
> It's not like it's really a big deal, and the profiler doesn't really
> points to it as a performance problem, but since those two arguments are
> so commonly used I think it'd be better for the method to know that than
> having thousands of exceptions thrown?
>
> --
> Gabriel Roldan
> grol...@opengeo.org
> Expert service straight from the developers
>
>
> ------------------------------------------------------------------------------
> WhatsUp Gold - Download Free Network Management Software
> The most intuitive, comprehensive, and cost-effective network
> management toolset available today. Delivers lowest initial
> acquisition cost and overall TCO of any competing solution.
> http://p.sf.net/sfu/whatsupgold-sd
> _______________________________________________
> Geotools-devel mailing list
> Geotools-devel@lists.sourceforge.net
> https://lists.sourceforge.net/lists/listinfo/geotools-devel
>
------------------------------------------------------------------------------
WhatsUp Gold - Download Free Network Management Software
The most intuitive, comprehensive, and cost-effective network
management toolset available today. Delivers lowest initial
acquisition cost and overall TCO of any competing solution.
http://p.sf.net/sfu/whatsupgold-sd
_______________________________________________
Geotools-devel mailing list
Geotools-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/geotools-devel
