POC Expose Pinot metrics #18581
POC Expose Pinot metrics #18581
Conversation
|
Thank you for your pull request and welcome to the Trino community. We require contributors to sign our Contributor License Agreement, and we don't seem to have you on file. Continue to work with us on the review and improvements in this PR, and submit the signed CLA to cla@trino.io. Photos, scans, or digitally-signed PDF files are all suitable. Processing may take a few days. The CLA needs to be on file before we merge your changes. For more information, see https://github.com/trinodb/cla |
|
Thank you for your pull request and welcome to the Trino community. We require contributors to sign our Contributor License Agreement, and we don't seem to have you on file. Continue to work with us on the review and improvements in this PR, and submit the signed CLA to cla@trino.io. Photos, scans, or digitally-signed PDF files are all suitable. Processing may take a few days. The CLA needs to be on file before we merge your changes. For more information, see https://github.com/trinodb/cla |
|
Let me close as stale PR. Please feel free to reopen if you still want to continue. |
Exposes Pinot Metrics
An issue with the above code is that it makes a separate call to Pinot just to get the metrics. Ideally we would have a single call to both:
a) construct the result data
b) construct metrics
Code is just for illustrative purposes / proof-of-concept.