Re: [Geotools-devel] [Geoserver-devel] Rendering pre process Mark Factories extension point proposal

2021-09-08 Thread Jody Garnett
Fernando: Rather than watching communication go by: - Moved your proposal to the description section (you had the proposal outlined as tasks) and wrote in my assumption for MarkFactoryProcessor interface. - Wrote up alternate proposal more formally -- Jody Garnett On Thu, 12 Aug 2021 at 14:34,

Re: [Geotools-devel] [Geoserver-devel] Rendering pre process Mark Factories extension point proposal

2021-09-08 Thread Jody Garnett
Frenando: I had a look again at your proposed interface, and it is not making sense to me, perhaps you could clarify: public interface MarkFactoryProcessorProvider { public static final Hints.Key MARK_FACTORY_PROCESSOR_PROVIDER_KEY = new

Re: [Geotools-devel] [Geoserver-devel] Rendering pre process Mark Factories extension point proposal

2021-09-08 Thread Fernando Mino
Hi Jody, thanks for your feedback. The problem with the alphabetical default on class names would be we will have SVG mark factory evaluated before WKT mark factories, which is the current performance killer scenario. Our backward compatibility strategy is to return the mark factory iterator as

Re: [Geotools-devel] [Geoserver-devel] Rendering pre process Mark Factories extension point proposal

2021-09-08 Thread Jody Garnett
Aside; do we need to make the default value for this hint alphabetical? To match prior tomcat behavior letting folks upgrade without configuration change. -- Jody Garnett On Wed, 8 Sept 2021 at 07:07, Jody Garnett wrote: > Thanks Fernando I appreciate you taking the time to revise the

Re: [Geotools-devel] [Geoserver-devel] Rendering pre process Mark Factories extension point proposal

2021-09-08 Thread Jody Garnett
Thanks Fernando I appreciate you taking the time to revise the proposal. +1 -- Jody Garnett On Mon, 6 Sept 2021 at 08:09, Fernando Mino < fernando.m...@geosolutionsgroup.com> wrote: > Thanks Jody for your hints. > > Indeed, using rendering hints to pass the pre-process behavior to GeoTools >