agavra opened a new pull request, #9513: URL: https://github.com/apache/pinot/pull/9513
fixes #9042 This PR refactors the query logging capabilities into its own log4j logger so that production servers can choose to configure that logging capability separately from the rest of the production logging. Example changes have been done in two `log4j.xml` files in this PR (please let me know if any other files need to update as well). As part of this refactor I also cleaned up the formatting of query logging so that adding new lines to the log isn't error-prone (which will also make it easier in the future if we want to add an option to log in JSON instead of a custom CSV format), and I fixed a race condition around logging the number of dropped queries. Lastly, I added test coverage :) cc @npawar -- 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] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
