[GitHub] leventov commented on issue #6559: Backpressure metric should not be emitted by default

[GitBox <gi...@apache.org>]

leventov commented on issue #6559: Backpressure metric should not be emitted by default
URL: https://github.com/apache/incubator-druid/issues/6559#issuecomment-435353510
 
 
   @jon-wei
   
   1) I strongly feel that no, no new per-*query* metric should be emitted by default whatsoever, and if we didn't have backwards compatibility constraints already, they should *all* have been opt-in. Explained why in #3954 and #4570. BTW interesting that right now, if somebody doesn't want the new metric, eg because their metric accepting system doesn't have extra capacity, there is no path to disable it other than writing a private extension.
   2) Above I suggested the schema where no private extension needed, only 2 extra config lines.

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
users@infra.apache.org


With regards,
Apache Git Services

---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@druid.apache.org
For additional commands, e-mail: commits-help@druid.apache.org