couchdb-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Damien Katz <dam...@apache.org>
Subject Re: [jira] Commented: (COUCHDB-397) POST _restart only restarts couch_server process
Date Mon, 29 Jun 2009 21:36:21 GMT

On Jun 29, 2009, at 4:42 PM, Adam Kocoloski (JIRA) wrote:

>
>    [ 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).

Yes, I think that's what I was trying to do.

>
> 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
>
>

Go for it. The point of _restart is for testing purposes, to make sure  
things are properly persisted and work properly after a restart.

-Damien

>
>> 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.
>


Mime
View raw message