nickva opened a new pull request #2444: Improve replicator error reporting
URL: https://github.com/apache/couchdb/pull/2444
 
 
   ### Overview
   
   Previously many HTTP requests failed noisily with `function_clause` errors.
   Expect some of those failures and handle them better. There are mainly 3 
types
   of improvements:
   
    1) Error messages are shorter. Instead of `function_clause` with a cryptic
    internal fun names, return a simple marker like `bulk_docs_failed`
   
    2) Include the error body if it was returned. HTTP failures besides the 
error
    code may contain useful information in the body to help debug the failure.
   
    3) Do not log or include the stack trace in the message. The error names are
    enough to identify the place were they are generated so avoid spamming the
    user and the logs with them. This is done by using `{shutdown, Error}` 
tuples
    to bubble up the error the replication scheduler.
   
   Fixes: https://github.com/apache/couchdb/issues/2413
   
   ### Testing recommendations
   
      `make eunit` 
   
   ### Checklist
   
   - [x] Code is written and works correctly
   - [x] Changes are covered by tests
   

----------------------------------------------------------------
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:
[email protected]


With regards,
Apache Git Services

Reply via email to