This change is fine by me, my original idea was to keep the API as
unaffected as possible and keep the dispatcher something running in the
background, optionally.
But - that means that the FileSystemRS also will need a
ResourceNotificationDispatcher.
The FileSystemWatcher almost implements it, but it takes an extra File
argument
addListener(File file, String path, ResourceListener listener)
Either that could be changed or a FileSystemDispatcher needs to be
wrapped around the FileSystemWatcher.
Regards
Niels
On 01/17/2016 05:39 PM, Jody Garnett wrote:
Niels - do you mind if I gather up the result of this email thread and
update the proposal - or would you like to?
On Sun, Jan 17, 2016 at 4:05 PM Ben Caradoc-Davies <b...@transient.nz
<mailto:b...@transient.nz>> wrote:
Sorry, yes it was, be me. I mistakenly assumed that all the GSIP-136
changes were isolated to the last two commits of PR #1361, but the
addition of this interface and the implementation
SimpleResourceNotificationDispatcher were included in an earlier
commit.
We will need to refactor it to match the final decision on GSIP-136.
Kind regards,
Ben.
On 18/01/16 12:55, Jody Garnett wrote:
> Ben I noticed ResourceNotificationDispatcher
>
<https://github.com/geoserver/geoserver/blob/master/src/platform/src/main/java/org/geoserver/platform/resource/ResourceNotificationDispatcher.java>
> is
> now available on master - was that accidentally merged?
>
> --
> Jody Garnett
--
Ben Caradoc-Davies <b...@transient.nz <mailto:b...@transient.nz>>
Director
Transient Software Limited <http://transient.nz/>
New Zealand
--
--
Jody Garnett
------------------------------------------------------------------------------
Site24x7 APM Insight: Get Deep Visibility into Application Performance
APM + Mobile APM + RUM: Monitor 3 App instances at just $35/Month
Monitor end-to-end web transactions and take corrective actions now
Troubleshoot faster and improve end-user experience. Signup Now!
http://pubads.g.doubleclick.net/gampad/clk?id=267308311&iu=/4140
_______________________________________________
Geoserver-devel mailing list
Geoserver-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/geoserver-devel
------------------------------------------------------------------------------
Site24x7 APM Insight: Get Deep Visibility into Application Performance
APM + Mobile APM + RUM: Monitor 3 App instances at just $35/Month
Monitor end-to-end web transactions and take corrective actions now
Troubleshoot faster and improve end-user experience. Signup Now!
http://pubads.g.doubleclick.net/gampad/clk?id=267308311&iu=/4140
_______________________________________________
Geoserver-devel mailing list
Geoserver-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/geoserver-devel
