Catching email list up on sprint discussion :) This conversation has
settled down and I will update the proposal and ask for a second review.
The take home message is:
- Watching resources change (on the file system or in jdbc resource store)
is different from broadcasting the change to listeners
- Our FileSystemWatcher should be updated/replaced with the Java 7
FileSystemWatcher
On Mon, 18 Jan 2016 at 09:39 Niels Charlier <ni...@scitus.be> wrote:
> 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>
> 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>
>> 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
> listGeoserver-devel@lists.sourceforge.nethttps://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
>
--
--
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
