chewbranca opened a new pull request, #4990:
URL: https://github.com/apache/couchdb/pull/4990

   ## Overview
   
   This PR includes the database name in the `"mango-stats"` alongside the 
existing data around documents processed to generate the request. The 
motivation here is two fold: 1) there's a scenario where `chttpd:maybe_log` 
returns false and the chttpd socket handling the request has died where a front 
load balancer processing the request never got the `X-Couch-Request-ID`, 
thereby losing the connection of the underlying request that induced the 
particular load; this is especially problematic when the socket died because 
the request was excessively long. 2) this allows for grouping by dbname 
directly on the reports, making it easier to quickly scan and summarize 
activity from the report.
   
   ## Testing recommendations
   
   I've added a new test to show the `dbname` field arrives in the 
`couch_log:report` logic yet does not extend the field into the http response 
when the `_find` query does `execution_stats=true`.
   
   ## Checklist
   
   - [X] Code is written and works correctly
   - [X] Changes are covered by tests
   - [ ] Any new configurable parameters are documented in 
`rel/overlay/etc/default.ini`
   - [ ] Documentation changes were made in the `src/docs` folder
   - [ ] Documentation changes were backported (separated PR) to affected 
branches
   


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

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to