On Fri, Jun 17, 2011 at 3:39 PM, Jody Garnett <[email protected]> wrote: > > Hi Chris: > > Andrea has been politely helping clean up some of the rendering code in > GeoTools; but has not really asked in a blunt enough fashion on the > geoserver-devel list. > > The KML code is going to be need in of some serious QA and testing after this > patch; do you have any developers who would be good for the work? We could > arrange an IRC breakout session (and I could be available to answer any api > change questions). > > I don't want to see this patch go ahead; and you learn about a sudden drop in > quality when taking GeoServer into production at the end of this release > cycle.
"Death of kml" is of course exxagerated, but in fact I could only check that the patch attached to http://jira.codehaus.org/browse/GEOS-4597 builds, I did not attempt to use any of the many KML output forms, options and whatnot. The code coverage is not high and I'm confident we'll see some ClassCastException in some code path in KML when the code tries to use a layer as a vector layer (because it needs a feature source and a style) when the old code chewed raster layers the same way anyways (I tried to pay attention, but it's a maze in there, not sure what all the possible call paths are) So it would really need someone to check and fix. I'm not volunteering to be that person. Cheers Andrea -- ------------------------------------------------------- Ing. Andrea Aime GeoSolutions S.A.S. Tech lead Via Poggio alle Viti 1187 55054 Massarosa (LU) Italy phone: +39 0584 962313 fax: +39 0584 962313 http://www.geo-solutions.it http://geo-solutions.blogspot.com/ http://www.youtube.com/user/GeoSolutionsIT http://www.linkedin.com/in/andreaaime http://twitter.com/geowolf ------------------------------------------------------- ------------------------------------------------------------------------------ EditLive Enterprise is the world's most technically advanced content authoring tool. Experience the power of Track Changes, Inline Image Editing and ensure content is compliant with Accessibility Checking. http://p.sf.net/sfu/ephox-dev2dev _______________________________________________ Geoserver-devel mailing list [email protected] https://lists.sourceforge.net/lists/listinfo/geoserver-devel
