From: Jody G. <jod...@gm...> - 2011-05-07 01:32:29
|
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 > gr...@op... > 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 > Geo...@li... > https://lists.sourceforge.net/lists/listinfo/geotools-devel > |