Hard to remember what they were for, if they are not in use we should remove.
On Sun, Nov 25, 2018 at 8:10 AM Andrea Aime <[email protected]> wrote: > Hi, > I believe that at some point in the past Jody and his colleagues worked on > some scalability enhancement > for the referencing subsystem that created a bunch of *Mediator classes, > in referencing and implementation > of thereof. The classes were merged, but never enabled, and I believe, > never actually put in use. > Do you know if there is anything actually using them? > > I have not thought much of them until now, but today I was looking at > referencing and some upgrades > (commons-pool in particular) and these classes got in the way of the > upgrade, making it more complicated. > > Coincidentally I was also looking at some scalability issue in > referencing, but found that today's use of > referencing goes though CRS.decode 99% of the time, so the easy way to > remove the issue is to put a thread safe > cache in CRS itself, without going through all the complication that the > mediators add to the mix. > > Long story short... if there is no use for them, can we please remove this > large bunch of classes from the > codebase, making it a bit faster to build, and a bit easier to understand > (at least, one will not get lost > looking into unused classes). > > 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 > <https://maps.google.com/?q=Via+di+Montramito+3/A%0D%0A55054++Massarosa&entry=gmail&source=g> > (LU) phone: +39 0584 962313 fax: +39 0584 1660272 mob: +39 339 8844549 > http://www.geo-solutions.it http://twitter.com/geosolutions_it > ------------------------------------------------------- *Con riferimento > alla normativa sul trattamento dei dati personali (Reg. UE 2016/679 - > Regolamento generale sulla protezione dei dati “GDPR”), si precisa che ogni > circostanza inerente alla presente email (il suo contenuto, gli eventuali > allegati, etc.) è un dato la cui conoscenza è riservata al/i solo/i > destinatario/i indicati dallo scrivente. Se il messaggio Le è giunto per > errore, è tenuta/o a cancellarlo, ogni altra operazione è illecita. Le > sarei comunque grato se potesse darmene notizia. This email is intended > only for the person or entity to which it is addressed and may contain > information that is privileged, confidential or otherwise protected from > disclosure. We remind that - as provided by European Regulation 2016/679 > “GDPR” - copying, dissemination or use of this e-mail or the information > herein by anyone other than the intended recipient is prohibited. If you > have received this email by mistake, please notify us immediately by > telephone or e-mail.* > _______________________________________________ > GeoTools-Devel mailing list > [email protected] > https://lists.sourceforge.net/lists/listinfo/geotools-devel > -- -- Jody Garnett
_______________________________________________ GeoTools-Devel mailing list [email protected] https://lists.sourceforge.net/lists/listinfo/geotools-devel
