Hi Jody.

> Hi Nicolai, I just found your article and loved it :)

Thanks! :)

> Any chance we could repost on the geotools blog, when we actually
> merge the PR :)

Only if we get this merged until end of the year. ;P

 so long ... Nicolai



On 07.11.2017 01:00, Jody Garnett wrote:
> Hi Nicolai, I just found your article (https://medium.com/codefx- 
> weekly/geotools-on-java-9-e8717e347a1f) and loved it :)
> 
> Any chance we could repost on the geotools blog, when we actually
> merge the PR :)
> 
> -- Jody Garnett
> 
> On 4 October 2017 at 11:21, Nicolai Parlog <[email protected]>
> wrote:
> 
>> Hi Andrea,
>> 
>> thanks for your feedback and for pointing out those projects - I 
>> didn't know about them. I did a quick search [1, 2] and only
>> found these two lines of code:
>> 
>> [3]:    Iterator<DXFWriter> it = writerRegistry 
>> .getServiceProviders(DXFWriter.class, null, null);
>> 
>> [4]:    Iterator i = FactoryRegistry 
>> .lookupProviders(extensionPoint);
>> 
>> Both are very easy to update. If these are all, I will include
>> them in the proposal section on "API before/after" and prepare a
>> PR that can be merged immediately after #1670 is merged.
>> 
>> so long ... Nicolai
>> 
>> 
>> [1]: https://github.com/GeoWebCache/geowebcache/search?utf8=%E2% 
>> 9C%93&q=FactoryRegistry&type= [2]: 
>> https://github.com/geoserver/geoserver/search?utf8=%E2%9C%93 
>> &q=FactoryRegistry&type= [3]: 
>> https://github.com/geoserver/geoserver/blob/55c4740060eb2b88 
>> 09ae935a510ac83eeb923313/src/extension/dxf/core/src/main/ 
>> java/org/geoserver/wfs/response/dxf/DXFWriterFinder.java#L58 
>> [4]: 
>> https://github.com/geoserver/geoserver/blob/55c4740060eb2b88 
>> 09ae935a510ac83eeb923313/src/platform/src/main/java/org/ 
>> geoserver/platform/GeoServerExtensions.java#L166
>> 
>> 
>> 
>> On 04.10.2017 10:19, Andrea Aime wrote:
>>> Hi Jody, I don't see a section talking about the directly
>>> affected downstream projects.
>>> 
>>> GeoWebCache and GeoServer build off the GeoTools snapshots 
>>> directly, so the moment the pull request land they will be hit 
>>> immediately, if that breaks the build, development will stop
>>> on both until we get the build back.
>>> 
>>> So I would like to see some section talking about that, if
>>> tests have been made, and in case of failure, what resources
>>> have been allocated to handle the switch. Doing so we could
>>> also learn things that might need changing in the proposal to
>>> make the switch less painful for other downstream projects.
>>> 
>>> Other downstream projects will eventually be affected, like 
>>> GeoMesa, GeoWave, GeoTrellis (and uDig too, the day that it
>>> manages to realign itself with GeoTools), not right out of the
>>> box I believe (unless they also depend on snapshots, I'm not
>>> sure), so having a migration guide in the mix would be good
>>> too.
>>> 
>>> I'm happy to provide some help during weekends.
>>> 
>>> Cheers Andrea
>>> 
>>> On Tue, Oct 3, 2017 at 11:31 PM, Jody Garnett 
>>> <[email protected]> wrote:
>>> 
>>>> As a follow up to today's meeting, please consider and vote
>>>> on the following proposal:
>>>> 
>>>> https://github.com/geotools/geotools/wiki/FactoryRegistry- 
>>>> Refactoring-for-Java-9-Compatibility
>>>> 
>>>> 
>>>> -- Jody Garnett
>>>> 
>>>> ------------------------------------------------------------ 
>>>> ------------------ Check out the vibrant tech community on
>>>> one of the world's most engaging tech sites, Slashdot.org! 
>>>> http://sdm.link/slashdot 
>>>> _______________________________________________
>>>> GeoTools-Devel mailing list
>>>> [email protected] 
>>>> https://lists.sourceforge.net/lists/listinfo/geotools-devel
>>>> 
>>>> 
>>> 
>>> 
>>> 
>>> 
>>> ------------------------------------------------------------
>> ------------------
>>> 
>>> 
>> Check out the vibrant tech community on one of the world's most
>>> engaging tech sites, Slashdot.org! http://sdm.link/slashdot
>>> 
>>> 
>>> 
>>> _______________________________________________ GeoTools-Devel 
>>> mailing list [email protected] 
>>> https://lists.sourceforge.net/lists/listinfo/geotools-devel
>>> 
>> 
>> --
>> 
>> PGP Key: 
>> http://keys.gnupg.net/pks/lookup?op=vindex&search=0xCA3BAD2E9CCCD509
>>
>>
>> 
Web:
>> http://codefx.org a blog about software development 
>> https://www.sitepoint.com/java high-quality Java/JVM content 
>> http://do-foss.de Free and Open Source Software for the City of
>> Dortmund
>> 
>> Twitter: https://twitter.com/nipafx
>> 
>> ------------------------------------------------------------ 
>> ------------------ Check out the vibrant tech community on one of
>> the world's most engaging tech sites, Slashdot.org!
>> http://sdm.link/slashdot 
>> _______________________________________________ GeoTools-Devel
>> mailing list [email protected] 
>> https://lists.sourceforge.net/lists/listinfo/geotools-devel
>> 
> 

-- 

PGP Key:
    http://keys.gnupg.net/pks/lookup?op=vindex&search=0xCA3BAD2E9CCCD509

Web:
    http://codefx.org
        a blog about software development
    https://www.sitepoint.com/java
        high-quality Java/JVM content
    http://do-foss.de
        Free and Open Source Software for the City of Dortmund

Twitter:
    https://twitter.com/nipafx

------------------------------------------------------------------------------
Check out the vibrant tech community on one of the world's most
engaging tech sites, Slashdot.org! http://sdm.link/slashdot
_______________________________________________
GeoTools-Devel mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/geotools-devel

Reply via email to