You are viewing a plain text version of this content. The canonical link for it is here.

Posted to commits@pulsar.apache.org by GitBox <gi...@apache.org> on 2022/09/01 07:58:27 UTC

[GitHub] [pulsar] asafm commented on pull request #16888: [improve][broker] Add gzip compression support for /metrics endpoint

asafm commented on PR #16888:
URL: https://github.com/apache/pulsar/pull/16888#issuecomment-1233894144

   > > > Can we enable conditional compression that only compresses payloads above a certain size?
   > > 
   > > 
   > > Yes, that is supported in GzipHandler.
   > > https://www.eclipse.org/jetty/javadoc/jetty-9/org/eclipse/jetty/server/handler/gzip/GzipHandler.html#setMinGzipSize(int)
   > 
   > That seems like a good indication to me that we should just enable it for all HTTP endpoints.
   
   One interesting question here is if we experience any issues currently with other HTTP endpoints. In Prometheus, the shear response size of 300 MB is worth compressing since the issue with a timeout. I wonder which other endpoints have similar issues?


-- 
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-unsubscribe@pulsar.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org