[ 
https://issues.apache.org/jira/browse/COUCHDB-397?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12725336#action_12725336
 ] 

Adam Kocoloski commented on COUCHDB-397:
----------------------------------------

Hi Paul, I probably shouldn't have brought up the whole exit(couch_config, 
blah) thing.  Let's focus on the supervisor behavior.

The trunk code *does not* restart any secondary services.  AFAICT these lines 
are a no-op:

    supervisor:terminate_child(couch_secondary_services, couch_server), 
    supervisor:restart_child(couch_secondary_services, couch_server). 

There is no child of couch_secondary_services named couch_server.  Damien 
should comment on the rationale.  Perhaps what was intended was

    supervisor:terminate_child(couch_server_sup, couch_secondary_services), 
    supervisor:restart_child(couch_server_sup, couch_secondary_services).

That change would also fix COUCHDB-398 -- it just restarts a bit more of the 
application than what I posted.

Not sure where I made a comment that indicated my diff in COUCHDB-398 didn't 
work.  I've checked -- it does reset the stats counters, and the test suite 
does pass.

I think having a working _restart handler would actually be kinda useful, much 
like the _log handler Jan implemented a while back.

I propose that /_restart should gracefully restart the entire couch 
application, not just the couch_server process.  Cheers, Adam



> POST _restart only restarts couch_server process
> ------------------------------------------------
>
>                 Key: COUCHDB-397
>                 URL: https://issues.apache.org/jira/browse/COUCHDB-397
>             Project: CouchDB
>          Issue Type: Bug
>    Affects Versions: 0.9
>            Reporter: Adam Kocoloski
>            Assignee: Adam Kocoloski
>
> Perhaps this is by design, but POSTing to /_restart only restarts the 
> couch_server process.  It does not restart any of the other myriad processes 
> CouchDB relies on for operation.  In particular, this causes the stats test 
> suite to occasionally fail, because the stats counters do not get reset.  The 
> test that fails is
> Assertion 'open_databases > 0 && max >= open_databases, name' failed: should 
> keep the same number of open databases when reaching the max_dbs_open limit

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.

Reply via email to