Re: [PR] Add dbname to mango exec stats [couchdb]

["chewbranca (via GitHub)" <gi...@apache.org>]

chewbranca commented on code in PR #4990:
URL: https://github.com/apache/couchdb/pull/4990#discussion_r1511896911


##########
src/mango/src/mango_execution_stats.hrl:
##########
@@ -16,5 +16,6 @@
     totalQuorumDocsExamined = 0,
     resultsReturned = 0,
     executionStartTime,
-    executionTimeMs
+    executionTimeMs,
+    dbname

Review Comment:
   Hmmm... I went with `dbname` because it's the primary name used in the codebase to track the name of the db. This module here is the exception, we don't normally utilize camel case in record definitions, so I'm not a big fan of spelling out `databaseName`.



-- 
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: notifications-unsubscribe@couchdb.apache.org

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