You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@pinot.apache.org by GitBox <gi...@apache.org> on 2020/08/16 01:04:23 UTC
[GitHub] [incubator-pinot] buchireddy opened a new issue #5871: Expose response stats in Pinot java client's BrokerReponse
buchireddy opened a new issue #5871:
URL: https://github.com/apache/incubator-pinot/issues/5871
Currently, response stats like `numDocsScanned`, `totalDocs`, and `timeUsedMs` are returned in the JSON response body but they're not exposed to the client in https://github.com/apache/incubator-pinot/blob/master/pinot-clients/pinot-java-client/src/main/java/org/apache/pinot/client/BrokerResponse.java
Exposing the response stats would allow the client application to log the status for slow queries and make the debugging easy instead of trying to reproduce the slow queries again.
I will raise a PR for this unless people have any comments.
cc @kishoreg
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@pinot.apache.org
For additional commands, e-mail: commits-help@pinot.apache.org
[GitHub] [incubator-pinot] buchireddy closed issue #5871: Expose response stats in Pinot java client's BrokerReponse
Posted by GitBox <gi...@apache.org>.
buchireddy closed issue #5871:
URL: https://github.com/apache/incubator-pinot/issues/5871
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@pinot.apache.org
For additional commands, e-mail: commits-help@pinot.apache.org
[GitHub] [incubator-pinot] buchireddy commented on issue #5871: Expose response stats in Pinot java client's BrokerReponse
Posted by GitBox <gi...@apache.org>.
buchireddy commented on issue #5871:
URL: https://github.com/apache/incubator-pinot/issues/5871#issuecomment-683166530
Yes. The PR is merged.
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@pinot.apache.org
For additional commands, e-mail: commits-help@pinot.apache.org
[GitHub] [incubator-pinot] kishoreg commented on issue #5871: Expose response stats in Pinot java client's BrokerReponse
Posted by GitBox <gi...@apache.org>.
kishoreg commented on issue #5871:
URL: https://github.com/apache/incubator-pinot/issues/5871#issuecomment-683042498
@buchireddy can we close this?
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@pinot.apache.org
For additional commands, e-mail: commits-help@pinot.apache.org