You are viewing a plain text version of this content. The canonical link for it is here.

Posted to notifications@couchdb.apache.org by GitBox <gi...@apache.org> on 2021/04/28 17:02:23 UTC

[GitHub] [couchdb] nickva opened a new pull request #3537: Install only one couch log error handler

nickva opened a new pull request #3537:
URL: https://github.com/apache/couchdb/pull/3537


   Previously we installed two of them, one through the `error_logger:add_report_handler/1` call and another through the `gen_event:add_sup_handler/3`. The first one was to setup `logger` handler, but the second was was still needed so we're notified if event manager died.
   
   The fix is to do what `error_logger:add_handler/1` does [1] and instead of calling the `gen_event:add_handler/3`, call `gen_server:add_sup_handler/3`.
   
   [1] [error_logger.erl#L453-L455](https://github.com/erlang/otp/blob/40922798411c2d23ee8a99456f96d6637c62b762/lib/kernel/src/error_logger.erl#L453-L455)
   
   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

[GitHub] [couchdb] nickva merged pull request #3537: Install only one couch log error handler

Posted by GitBox <gi...@apache.org>.

nickva merged pull request #3537:
URL: https://github.com/apache/couchdb/pull/3537


   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org