chewbranca commented on PR #5602: URL: https://github.com/apache/couchdb/pull/5602#issuecomment-3127712922
Alright I've gone through and addressed most things: * I've reworked the docs to with more details on how/where/when/why to utilize CSRT and also moved them into the Sphinx docs in: 828667e3d as well as introducing a new example or two * In eb92a4d5a I reworked `csrt_query:group_by` to use the more efficient `ets:select` as well as to utilize that to enforce the `query_cardinality_limit()` constraints which I also updated to be more clear in that commit. I also fixed a small bug in `csrt_query:options/1` * I added default Logger Matchers `all_coordinators` and `all_rpc_workers` in 00a02af01 so we'd have an easy and dedicated way to toggle on logging of all coordinate process lifecycle reports, and if so desired, the same with `all_rpc_workers`, but the other impact of this change is that we can now easily aggregate on only coordinators or workers when querying against the HTTP and query API * updated `couch_log_formatter:format_meta` to ignore `null` values so we don't end up with report entries like `dbname="null"` * cleaned up various test cases based on PR beedback in 1380b09fc * various other cleanup bits -- 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]
