themarcelor opened a new issue #12112:
URL: https://github.com/apache/druid/issues/12112


   Can you please augment the jetty logger to also present the HTTP response 
code? 🙏🏼 
   
   We just need to add `response.getStatus()` to this class:
   
   ```public class JettyRequestLog extends AbstractLifeCycle implements 
RequestLog
   {
     private static final Logger logger = new 
Logger("org.apache.druid.jetty.RequestLog");
   
     @Override
     public void log(Request request, Response response)
     {
       if (logger.isDebugEnabled()) {
         logger.debug(
             "%s %s %s %s %s",
             request.getRemoteAddr(),
             request.getMethod(),
             request.getHttpURI().toString(),
             request.getProtocol(),
             response.getStatus()
         );
       }
     }
   }
   ```
   This would be extremely helpful to calculate the SR (Success Rate) based on 
the percentage of non-http-5xx requests.


-- 
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]

Reply via email to