That is great - which book? I trust your intelligence on a wide range of matters, while you research the technical side - there are a couple strategic decisions in the middle of this proposal that I would appreciate your guidance / input on.
1. remove org.opengis packages It seems like good timing, and with the module system any conflict over org.opengis packages will be more keenly felt. Here is the proposal renaming org.opengis to org.geotools.api: - org.geotools.api.filter.Filter - org.geotools.api.filter.PropertyIsEqualTo - org.geotools.filter.FilterAbstract - org.geotools.filter.IsEqualsToImpl 2. folding gt-cql together gt-main ECQL is now a well established stable part of our library, moving it closer should allow us to use it in more places helping with ease of use. 3. Priority to change packages, but strictly not change interface / class names Goal is to allow clients to confidently fix their imports using "organize imports" or similar. While keeping this goal we *could* fold most interfaces back into place: - org.geotools.filter.Filter - org.geotools.filter.PropertyIsEqualTo - org.geotools.filter.FilterAbstract - org.geotools.filter.IsEqualsToImpl However the style interfaces narrowly avoid conflict: - org.geotools.style.PolygonSymbolizer - readonly - org.geotools.styling.PolygonSymbolizer - mutable - org.geotools.styling.PolygonSymbolizerImpl If we relaxed the "no changing classnames" guidelines we could gather these into a single package: - org.geotools.style.PolygonSymbolizer - readonly - org.geotools.style.PolygonSymbolizer2 - mutable - org.geotools.style.PolygonSymbolizerImpl 4. Priority to core library, plugins are expected to repackage I am pretty content with this direction, the majority of plugins have their own distinct package. I am not sure if more sensitivity is required here? -- Jody Garnett On Wed, 19 Sep 2018 at 09:49, Andrea Aime <andrea.a...@geo-solutions.it> wrote: > Hi Jody, > I got myself a book on java modularity and reading through it in my spare > time. > Will take a couple of weeks I believe before I can add any intelligent > comment on the proposal, sorry :-) > > Cheers > Andrea > > On Tue, Sep 18, 2018 at 1:18 AM Jody Garnett <jody.garn...@gmail.com> > wrote: > >> Proposal is here: Restructure GeoTools into Jigsaw modules >> <https://github.com/geotools/geotools/wiki/Restructure-GeoTools-into-Jigsaw-modules> >> >> Feedback welcome, I expect we will have some discussion before voting. >> -- >> Jody Garnett >> _______________________________________________ >> GeoTools-Devel mailing list >> GeoTools-Devel@lists.sourceforge.net >> https://lists.sourceforge.net/lists/listinfo/geotools-devel >> > > > -- > > Regards, Andrea Aime == 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 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 GeoTools-Devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/geotools-devel
