praveenc7 commented on PR #15585: URL: https://github.com/apache/pinot/pull/15585#issuecomment-2822985353
> lgtm, here are some suggestions for the metric names `DEEP_STORE_WRITE_BYTES_IN_PROGRESS DEEP_STORE_WRITE_OPS_IN_PROGRESS DEEP_STORE_READ_OPS_IN_PROGRESS DEEP_STORE_READ_BYTES_IN_PROGRESS DEEP_STORE_TOTAL_READ_BYTES DEEP_STORE_TOTAL_WRITE_BYTES` You can also choose to abbreviate `DEEP_STORE` as `DS` if you like. Or, `DEEP_STORE` Thanks @mcvsubbu , like `DEEP_STORE_WRITE_BYTES_IN_PROGRESS` | `DEEP_STORE_WRITE_OPS_IN_PROGRESS` format, updated this. Would like to keep `DEEP_STORE` for clarity -- 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
