jtuglu1 commented on PR #18631: URL: https://github.com/apache/druid/pull/18631#issuecomment-3403045550
I found some gaps with no query/time metric emission for various failure scenarios (403 auth, sql query unexpected arg, etc.) during this PR which I think can be followed up in a subsequent change. This is for two reasons: 1. IMO these changes are logically separate and shouldn't be coupled with each other. 1. Reduce the diff to make it easier to backport this commit in case it doesn't make it into v35. -- 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]
