On Mon, Jan 2, 2012 at 1:06 PM, Gabriel Roldan <grol...@opengeo.org> wrote:
> On Mon, Jan 2, 2012 at 2:33 PM, Andrea Aime
> <andrea.a...@geo-solutions.it> wrote:
> > On Mon, Jan 2, 2012 at 6:22 PM, Justin Deoliveira <jdeol...@opengeo.org>
> > wrote:
> >>
> >> What about just having JAIInfo.jai be lazily evaluated, and have it just
> >> return the JAI singleton directly. JAI.getDefaultInstance() doesn't
> seem to
> >> be an expensive lookup at all (just returns a static singleton), so
> having
> >> it be called continuously should not be an issue.
>
> You mean JAIInfoImpl.getJAI(){ return JAI.getDefaultInstance();} ?
>
> Guess it'd work. Still I'd be up to avoiding any non configuration
> object as part of the configuration API. But feel free to implement it
> that way or give me the go ahead and I'll do it myself. I suppose the
> change should come along with the deprecation of JAIInfo.setJAI(JAI)?
>
Yeah, I try to avoid it as well... , but that was how the interface was
designed from day one so seems the path of least resistance. The api does
employ for things like derived properties and such that aren't strictly
"configuration" or plain regular properties.
Feel free to proceed how you see fit. I guess I would say change getJAI()
as we discussed directly on JAIInfo. As for deprecating setJAI()... i am
fine with that as well... although we could just leave
the property settable. A third alternative would be to make it a derived
properly, deprecate both getJAI() and setJAI() and replace with jai().
Maybe in the interest of you moving forward just update getJAI() and we
discuss the rest further? Up to you.
>
> Cheers,
> Gabriel
>
> >
> >
> > Agreed. However I guess we should setup some rule about what *Info object
> > are allowed to keep state (as a field)
> > and which ones cannot
> >
> > Cheers
> > Andrea
>
--
Justin Deoliveira
OpenGeo - http://opengeo.org
Enterprise support for open source geospatial.
------------------------------------------------------------------------------
Ridiculously easy VDI. With Citrix VDI-in-a-Box, you don't need a complex
infrastructure or vast IT resources to deliver seamless, secure access to
virtual desktops. With this all-in-one solution, easily deploy virtual
desktops for less than the cost of PCs and save 60% on VDI infrastructure
costs. Try it free! http://p.sf.net/sfu/Citrix-VDIinabox
_______________________________________________
Geoserver-devel mailing list
Geoserver-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/geoserver-devel
