Hi Kevin and Gabriel,

I have a few questions regarding the jdbc resource store and the 
resource store API in general that I'd like to hear your opinion on. 
Including the mailing list because others might have thoughts on this.

1) The ResourceStore does not appear to provide anything for creating 
directories. At the moment GeoServerResourceLoader.getOrCreateDirectory 
is used for that. How can we change all modules to use the generic 
Resource Store rather than the file based one directly without such a 
method in the API?

2)  In your implementation, the JDBCResource is using the OID primary 
key as a handle. At first I thought that made sense but I think there 
are some issues which respect to how the API assumes resources to work. 
Imagine that somebody else (possibly a different geoserver instance, as 
this is meant for clustering) overwrites the file or deletes it and then 
recreates one at the same location. Your resource holding the OID handle 
will be out of date. It might be able to notice the delete, but not the 
new content. But if you for example look at the rename test theory, it 
expects that both resources have changed to reflect the new reality. 
Many other parts of the API and the tests suggest that the ResourceStore 
is meant to support concurrent access.

The way that File System resources work is that they are simply wrapping 
around a File which is basically just wrapping around String which holds 
the path, without any state stored. Each method will query the file 
system using that path. My idea is that the only way for the JDBCStore 
to have the same expected behaviour is to also use the full path as the 
handle and in each method query the database  using the full path. Does 
that make sense? Do you agree with my conclusion?

Kind Regards
Niels

------------------------------------------------------------------------------
Don't Limit Your Business. Reach for the Cloud.
GigeNET's Cloud Solutions provide you with the tools and support that
you need to offload your IT needs and focus on growing your business.
Configured For All Businesses. Start Your Cloud Today.
https://www.gigenetcloud.com/
_______________________________________________
Geoserver-devel mailing list
Geoserver-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/geoserver-devel

Reply via email to