[GitHub] [hbase] caroliney14 edited a comment on pull request #3535: HBASE-25469 Create RIT servlet in HMaster to track more detailed RIT info not captured in metrics

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

caroliney14 edited a comment on pull request #3535:
URL: https://github.com/apache/hbase/pull/3535#issuecomment-891412977


   @bharathv there already are multiple places where RIT info can be found: the main `/master-status` page and the `rits.jsp` page. I don't think it's that confusing what the differences between the different RIT info sources are; the `/master-status` gives an overview of the RIT, the `rits.jsp` gives more details in a human-friendly format, and the `/rit` servlet displays the metrics we want as JSON. there are indeed overlaps between these places, but each has its own purpose that is independent of the other ones. we may choose to display different info in the JSON than on the `rits.jsp` page (I asked for feedback on [this JIRA comment](https://issues.apache.org/jira/browse/HBASE-25469?focusedCommentId=17388214&page=com.atlassian.jira.plugin.system.issuetabpanels%3Acomment-tabpanel#comment-17388214) about whether we should consider displaying only RITs over threshold in the JSON, or consider grouping by server name or state in the JSON -- if you or @apurtell have any thoughts there,
  please do share). that is just my personal opinion though, any thoughts @apurtell?


-- 
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: issues-unsubscribe@hbase.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org