gianm commented on a change in pull request #7562: Enable ability to toggle SegmentMetadata request logging on/off URL: https://github.com/apache/incubator-druid/pull/7562#discussion_r302716748
########## File path: docs/content/configuration/index.md ########## @@ -316,6 +316,7 @@ All processes that can serve queries can also log the query requests they see. B |Property|Description|Default| |--------|-----------|-------| |`druid.request.logging.type`|Choices: noop, file, emitter, slf4j, filtered, composing, switching. How to log every query request.|[required to configure request logging]| +| `druid.request.logging.logSegmentMetadataQueries`| Boolean indicating if SegmentMetadata Queries should be logged by the Request Logger.| true| Review comment: I think the way this property behaves is good, but it would be better inside FilteredRequestLogger. That way, it's composable, rather than needing to be implemented inside each and every RequestLogger. ---------------------------------------------------------------- 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 --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
