jugomezv commented on PR #10418: URL: https://github.com/apache/pinot/pull/10418#issuecomment-1478445090
> A new API work. Does it make sense to still merge this as is in the meantime? It's still a fairly new metric. @jadami10 As discussed earlier this patch will not be displaying correct times either: so the proposal is to modify batch interface to provide timestamp even for filtered messages, that way we will publish correct metrics even in the face of all filtered events. does that make sense? -- 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
