I can see your point, but I am personally not 100% convinced. I
understand that the files are not copied to a new location by the
importer module and may in fact well already be in the data directory (=
the resource store). On the other hand, it does regard data rather than
configurations and therefore should not be stored in the jdbcstore,
there I agree with you. BUT I thought it could theoretically be possible
in the future that the module is also used for configuration files that
the user does want to be stored in the jdbcstore. To make things easy,
the jdbcstore is equipped to ignore data files inside the data directory
and leave them on the hard disk. Therefore it is never really harmful in
any way to use resources instead of files, but it allows for more
options for the future with less duplicated code. This was the idea
behind it. Just my 2 cents.
Kind Regards
Niels
On 20-05-16 02:25, Torben Barsballe wrote:
While looking at Niels' GEOS-7513 fix
<https://github.com/geoserver/geoserver/pull/1601>, Jody noticed that
the ResourceStore work from earlier this year changed the Importer
module to use Resource for FileData. This is one place where se
/shouldn't/ be using resource, as FileData represents a file being
imported into GeoServer (i.e. not yet in GeoServer). Importer has a
seperate comparable object to deal with non-File import sources. I can
fix Importer to not use Resource in these places tommorow, but in the
mean time we should consider the GeoServer 2.9.0 release blocked.
In addition:
Kevin - are you able to do the GeoWebCachwe release for 1.9.0, given
your current injury? If not, are you available to do it Andrea? I
believe you have run through the GeoWebCache release before?
Torben
On Wed, May 18, 2016 at 2:53 PM, Jody Garnett <jody.garn...@gmail.com
<mailto:jody.garn...@gmail.com>> wrote:
Gathering hit list from yesterdays meeting and emails:
- Demo Page (GEOS-7513
<https://osgeo-org.atlassian.net/browse/GEOS-7513>): Niels is
looking at using a path (rather than a Resource) to fix the demo page.
- Demo Page (GEOS-6146
<https://osgeo-org.atlassian.net/browse/GEOS-6146>): nurc:img_sample2
disabled (either enable or put a comment in the demo request that
uses it)
- windows wrapper (GEOS-7507
<https://osgeo-org.atlassian.net/browse/GEOS-7507>): testing as
admin failed (editing wrapper.conf to match the branch), and we
know a win32 jre is needed
To be merged and backported:
- process factory available check (GEOT-5418
<https://osgeo-org.atlassian.net/browse/GEOT-5418>) - fixing
styling issues and merging/backport
- 1597 <https://github.com/geoserver/geoserver/pull/1597> imagemap fix
- 1589
<https://github.com/geoserver/geoserver/pull/1589> clustering
resource fix
Not included in release:
- checking use of resource vs file use
- ldap user/role service
- layer group virtual service fix
- tabs vs spaces
--
Jody Garnett
------------------------------------------------------------------------------
Mobile security can be enabling, not merely restricting. Employees who
bring their own devices (BYOD) to work are irked by the imposition
of MDM
restrictions. Mobile Device Manager Plus allows you to control
only the
apps on BYO-devices by containerizing them, leaving personal data
untouched!
https://ad.doubleclick.net/ddm/clk/304595813;131938128;j
_______________________________________________
Geoserver-devel mailing list
Geoserver-devel@lists.sourceforge.net
<mailto:Geoserver-devel@lists.sourceforge.net>
https://lists.sourceforge.net/lists/listinfo/geoserver-devel
------------------------------------------------------------------------------
Mobile security can be enabling, not merely restricting. Employees who
bring their own devices (BYOD) to work are irked by the imposition of MDM
restrictions. Mobile Device Manager Plus allows you to control only the
apps on BYO-devices by containerizing them, leaving personal data untouched!
https://ad.doubleclick.net/ddm/clk/304595813;131938128;j
_______________________________________________
Geoserver-devel mailing list
Geoserver-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/geoserver-devel
------------------------------------------------------------------------------
Mobile security can be enabling, not merely restricting. Employees who
bring their own devices (BYOD) to work are irked by the imposition of MDM
restrictions. Mobile Device Manager Plus allows you to control only the
apps on BYO-devices by containerizing them, leaving personal data untouched!
https://ad.doubleclick.net/ddm/clk/304595813;131938128;j
_______________________________________________
Geoserver-devel mailing list
Geoserver-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/geoserver-devel
