jadami10 opened a new issue, #11119: URL: https://github.com/apache/pinot/issues/11119
The broker query logs contain a field with `query=`. That field always has spaces, sometimes new lines, and potentially double quotes. This makes logging infra, like splunk, not parse it correctly as "key=value" Would we have any concerns with 1. double quoting the query 2. escaping double quotes in the query We have a [test](https://github.com/apache/pinot/blob/c32318360e46a4030bfbff9d11f57b7fa0564397/pinot-broker/src/test/java/org/apache/pinot/broker/querylog/QueryLoggerTest.java#L113) for this, so i'm not sure if folks are relying on this. But if so, this would be a backwards incompatible change. -- 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: commits-unsubscr...@pinot.apache.org.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: commits-unsubscr...@pinot.apache.org For additional commands, e-mail: commits-h...@pinot.apache.org
