albertobastos commented on PR #15264: URL: https://github.com/apache/pinot/pull/15264#issuecomment-2728467575
I see your point @mayankshriv. Ideally the sql query should only be logged the first time, and just use the requestId to trace it back during the second log. Was afraid of doing that in case someone is already consuming this logs and expects the sql to be included on the second line, but I guess they will have to adapt. Also, fixed that the change was only made for SSE. Now is also applied to MSE. Sorry for that. -- 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 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
