[
https://issues.apache.org/jira/browse/COUCHDB-3102?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15433820#comment-15433820
]
ASF GitHub Bot commented on COUCHDB-3102:
-----------------------------------------
GitHub user iilyak opened a pull request:
https://github.com/apache/couchdb-config/pull/12
Add config_listener_mon:start_link/2
This a fixup commit for f09a2eb7d
COUCHDB-3102
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/cloudant/couchdb-config
3102-fixup-for-config_subscription
Alternatively you can review and apply these changes as the patch at:
https://github.com/apache/couchdb-config/pull/12.patch
To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:
This closes #12
----
commit 144da209b5704539dec757a2c2070bb4b6525745
Author: ILYA Khlopotov <iil...@ca.ibm.com>
Date: 2016-08-23T23:01:48Z
Add config_listener_mon:start_link/2
This a fixup commit for f09a2eb7d
COUCHDB-3102
----
> The xxx_config_listener doesn't restart after a failure
> -------------------------------------------------------
>
> Key: COUCHDB-3102
> URL: https://issues.apache.org/jira/browse/COUCHDB-3102
> Project: CouchDB
> Issue Type: Bug
> Reporter: ILYA
>
> The
> [config_listener_mon|https://github.com/apache/couchdb-config/blob/master/src/config_listener_mon.erl#L44]
> monitors the caller.
> However in lots of places we use [following
> pattern|https://github.com/apache/couchdb-chttpd/blob/master/src/chttpd_config_listener.erl#L45:L49]
> {code}
> handle_config_terminate(_Server, _Reason, State) ->
> spawn(fun() ->
> timer:sleep(5000),
> config:listen_for_changes(?MODULE, State)
> end).
> {code}
> In this case the spawned fun exits right after the monitor has been
> established.
> Which means config_listener_mon would terminate. Effectively removing event
> handler from config_event process (gen_event event manager).
> When this happen any further configuration changes get unnoticed by processes.
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)
