That sounds fine Matt,

We woudl like to backport this work, so we may need to ask GeoServer 2.11.x
to use reflection to call this method in order not to backport an api
change.

--
Jody Garnett

On 13 April 2017 at 12:15, Matt Kruszewski <[email protected]>
wrote:

> Hi Andrea,
>
> Continuing with the idea of a generalized way to reset
> ExternalGraphicFactory caches, would something as simple as a
> CachingExternalGraphicFactory interface be appropriate? Something along the
> lines of this branch
> <https://github.com/mattkrusz/geotools/commit/632b8508c4992d211130cea7f53c156182553db3>?
> The GS lifecycle handler would then just reset the cache based on the
> interface. (I left the existing static resetCache() methods in place in
> this example for GeoServer's sake).
>
> - Matt
>
> On Fri, Mar 24, 2017 at 12:56 PM, Andrea Aime <
> [email protected]> wrote:
>
>> On Fri, Mar 24, 2017 at 6:05 PM, Matt Kruszewski <
>> [email protected]> wrote:
>>
>>> Hi all,
>>>
>>> I am working on an implementation of the ExternalGraphicFactory
>>> interface that supports Mapbox-style spritesheets (spec
>>> <https://www.mapbox.com/mapbox-gl-js/style-spec/#sprite>), and I have a
>>> question about image caching.
>>>
>>> I see that the current implementation of ImageGraphicFactory caches
>>> BufferedImages and provides a resetCache() method. That method is not part
>>> of the interface -- instead, GeoServer calls it directly in the
>>> WMSLifecycleHandler::onReset() method. Is that image-cache-reset
>>> functionality generalized anywhere that I am not seeing?
>>>
>>
>> Nope
>>
>>
>>> Do we want it to be?
>>>
>>
>> That would be a good idea imho
>>
>>
>>> Or, on the other hand, would it be reasonable for a style module to
>>> provide its own lifecycle handler and manage its cache that way?
>>>
>>
>> Imho a style module should not be aware of those details, they should be
>> orthogonal, a graphic factory can either be used
>> from all styling languages, or its design is broken
>>
>> Cheers
>> Andrea
>>
>>
>> --
>> ==
>> GeoServer Professional Services from the experts! Visit
>> http://goo.gl/it488V for more information.
>> ==
>>
>> Ing. Andrea Aime
>> @geowolf
>> Technical Lead
>>
>> GeoSolutions S.A.S.
>> Via di Montramito 3/A
>> 55054  Massarosa (LU)
>> phone: +39 0584 962313 <+39%200584%20962313>
>> fax: +39 0584 1660272 <+39%200584%20166%200272>
>> mob: +39  339 8844549 <+39%20339%20884%204549>
>>
>> http://www.geo-solutions.it
>> http://twitter.com/geosolutions_it
>>
>> *AVVERTENZE AI SENSI DEL D.Lgs. 196/2003*
>>
>> Le informazioni contenute in questo messaggio di posta elettronica e/o
>> nel/i file/s allegato/i sono da considerarsi strettamente riservate. Il
>> loro utilizzo è consentito esclusivamente al destinatario del messaggio,
>> per le finalità indicate nel messaggio stesso. Qualora riceviate questo
>> messaggio senza esserne il destinatario, Vi preghiamo cortesemente di
>> darcene notizia via e-mail e di procedere alla distruzione del messaggio
>> stesso, cancellandolo dal Vostro sistema. Conservare il messaggio stesso,
>> divulgarlo anche in parte, distribuirlo ad altri soggetti, copiarlo, od
>> utilizzarlo per finalità diverse, costituisce comportamento contrario ai
>> principi dettati dal D.Lgs. 196/2003.
>>
>>
>>
>> The information in this message and/or attachments, is intended solely
>> for the attention and use of the named addressee(s) and may be confidential
>> or proprietary in nature or covered by the provisions of privacy act
>> (Legislative Decree June, 30 2003, no.196 - Italy's New Data Protection
>> Code).Any use not in accord with its purpose, any disclosure, reproduction,
>> copying, distribution, or either dissemination, either whole or partial, is
>> strictly forbidden except previous formal approval of the named
>> addressee(s). If you are not the intended recipient, please contact
>> immediately the sender by telephone, fax or e-mail and delete the
>> information in this message that has been received in error. The sender
>> does not give any warranty or accept liability as the content, accuracy or
>> completeness of sent messages and accepts no responsibility  for changes
>> made after they were sent or for other risks which arise as a result of
>> e-mail transmission, viruses, etc.
>>
>> -------------------------------------------------------
>>
>
>
> ------------------------------------------------------------
> ------------------
> Check out the vibrant tech community on one of the world's most
> engaging tech sites, Slashdot.org! http://sdm.link/slashdot
> _______________________________________________
> Geoserver-devel mailing list
> [email protected]
> https://lists.sourceforge.net/lists/listinfo/geoserver-devel
>
>
------------------------------------------------------------------------------
Check out the vibrant tech community on one of the world's most
engaging tech sites, Slashdot.org! http://sdm.link/slashdot
_______________________________________________
Geoserver-devel mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/geoserver-devel

Reply via email to