> > Hum... wfs-ng was supposed to be a drop-in replacement, not having a
>> upgrade
>> > path seems to be a serious issue to me.
>> > Where is it documented?
>>
>> Quick search got us this:
>> https://osgeo-org.atlassian.net/browse/GEOT-4883
>> and we googled some more messages on it.
>
> A backwards incompable change blocking upgrades presented as a GeoServer
> requirement and resolved as "not a bug"...
>

Reading this issue, and I agree it is not a bug - the new datastore is
behaving as designed in order to preserve valid names for GML generation.

For this specific case the old datastore would of required a similar fix
(to prevent type names being of the form
"local_namespace:remote_namespace:typename").


> As far as I know GeoServer was already handling the ":" in the name, and
> the admin is able to rename layers as they see fit.
> Yep, there is code stripping the prefix:
>
> https://github.com/geoserver/geoserver/blob/master/src/main/src/main/java/org/geoserver/catalog/CatalogBuilder.java#L1212
>

GeoServer may have work arounds, but I do not mind if the new DataStore is
taking more care to produce consistent type names.

I'd suggest to make this name mapping optional and disable by default to
> preserve backwards compatibility.
>

The drop-in replacement metric is being a trouble here, care was taken to
allow this to drop in and use the same connection parameters. I would
understand if the data store has some differences.

Is this change visible to GeoServer users? I guess it may be if the
geotools provided type name is recorded in the geoeserver configuration ...
and that value is fixed (there is no way to edit it as I have discovered
when renaming tables in PostGIS).

Jody
------------------------------------------------------------------------------
Find and fix application performance issues faster with Applications Manager
Applications Manager provides deep performance insights into multiple tiers of
your business applications. It resolves application problems quickly and
reduces your MTTR. Get your free trial!
https://ad.doubleclick.net/ddm/clk/302982198;130105516;z
_______________________________________________
GeoTools-Devel mailing list
GeoTools-Devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/geotools-devel

Reply via email to