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/55c4740060eb2b8809ae935a510ac83eeb923313/src/extension/dxf/core/src/main/java/org/geoserver/wfs/response/dxf/DXFWriterFinder.java#L58
[4]:
https://github.com/geoserver/geoserver/blob/55c4740060eb2b8809ae935a510ac83eeb923313/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
