Hey folks,
With Niels last pull request for wfs-ng adding the requested tests I
believe all the feedback has been addressed regarding a transition to
wfs-ng. So I wanted to start hashing out the plan to migrate.
At the moment there are some core dependencies on the WFSDataStoreFactory
class from the old module, which lives in the package
org.geotools.data.wfs. Currently the ng factory lives in the package
org.geoserver.data.wfs.impl, which is kind of strange to have a public
interface in an impl package.
If we moved the ng factory into the data.wfs package at which point we
don't have to change a line of geoserver code in order to use it. However
this would mean that the new and the old datastore couldn't co-exist at the
same time.
A few options I can think of.
1. Allow the two modules to co-exist on the classpath. This approach I
think would be the most work because we would have to:
a. come up with a way for the factories to know about one another
b. make the code in geoserver dynamically access one or the other, or
continue to use the old datastore for the remove ows support, and the new
one for datastore access.
2. Make wfs-ng a compile time drop-in and package wfs-ng as an extension
and let the users know that if they want to use it they have to remove the
old jar first.
3. Rename wfs-ng to wfs (using a different group name to avoid a maven
conflict) and package as an extension. When the user installs it it will
overwrite the old version.
Thoughts?
--
*Justin Deoliveira*
Vice President, Engineering | Boundless
[email protected]
@j_deolive <https://twitter.com/j_deolive>
------------------------------------------------------------------------------
HPCC Systems Open Source Big Data Platform from LexisNexis Risk Solutions
Find What Matters Most in Your Big Data with HPCC Systems
Open Source. Fast. Scalable. Simple. Ideal for Dirty Data.
Leverages Graph Analysis for Fast Processing & Easy Data Exploration
http://p.sf.net/sfu/hpccsystems
_______________________________________________
Geoserver-devel mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/geoserver-devel
