Resending original as plain text. Jeffrey Wood Software Engineer Geocent
Email: [email protected] Ph (BR): 225-214-4346 From: Jeffrey Wood [mailto:[email protected]] Sent: Monday, November 04, 2013 11:06 AM To: [email protected] Subject: [Geotools-devel] Feedback requested - for pull request #305 - Add hint property (Map<String, String>) to GeneralParameterValue interface and AbstractParameter which implements it. GeoTools developers, Per Andrea, I am writing to this list re: our proposed change to the GeoTools public interface. We are proposing a change to the GeoTools GeneralParameterValue interface and the class that implements it, AbstractParameter, to add a property that is a Map <String, String>. Why add this property? To support our GeoServer change to let users set the layer's elevation dimension default value. Currently, GeoServer does not look for any layer input, it decides for itself to use the minimum numeric value. We need a way to transport the layer specific value, and the way we saw to do that is to provide it through the 'merged' parameter. If it is not provided, GeoServer uses its existing method. Why a Map? We did not want to make it a named property, we instead wanted to make a single Collection that could hold this layer attribute and any other layer attributes without further changes to the GeoTools entity. So we propose a Map<String, String> and the user can look up the value they are interested in. If you have suggestions or questions, please let me know. The GeoTools pull request: https://github.com/geotools/geotools/pull/305 The GeoServer pull request: https://github.com/geoserver/geoserver/pull/377 Thank you, Jeffrey Wood Software Engineer Geocent Email: [email protected] Ph (BR): 225-214-4346 From: Jeffrey Wood Sent: Monday, November 04, 2013 8:39 AM To: 'geotools/geotools'; geotools/geotools Subject: RE: [geotools] Add hint property (Map<String,String>) to GeneralParameterValue interface and AbstractParameter which implements it. (#305) Thank you Andrea, Yes, I will send an email to geotools-devel to start the discussion. Jeffrey Wood Software Engineer Geocent Email: [email protected] Ph (BR): 225-214-4346 From: Andrea Aime [mailto:[email protected]] Sent: Sunday, November 03, 2013 5:03 AM To: geotools/geotools Cc: Jeffrey Wood Subject: Re: [geotools] Add hint property (Map<String,String>) to GeneralParameterValue interface and AbstractParameter which implements it. (#305) Hi, you are changing a published interface, a discussion needs to take place on geotools-devel where you explain why you made that change, and why it was necessary to go down this path, or why it's better than possible alternate solutions. Can you start it? — Reply to this email directly or view it on GitHub. ------------------------------------------------------------------------------ Android is increasing in popularity, but the open development platform that developers love is also attractive to malware creators. Download this white paper to learn more about secure code signing practices that can help keep Android apps secure. http://pubads.g.doubleclick.net/gampad/clk?id=65839951&iu=/4140/ostg.clktrk _______________________________________________ GeoTools-Devel mailing list [email protected] https://lists.sourceforge.net/lists/listinfo/geotools-devel
