You are viewing a plain text version of this content. The canonical link for it is here.

Posted to notifications@accumulo.apache.org by GitBox <gi...@apache.org> on 2021/04/15 11:23:29 UTC

[GitHub] [accumulo] karthick-rn commented on pull request #2020: Remove ZK stats from Monitor overview page

karthick-rn commented on pull request #2020:
URL: https://github.com/apache/accumulo/pull/2020#issuecomment-820347829


   > The removal of the `refreshZKTable` function from `overview.js` and the place where that function is called can also be included in this PR, and any other related javascript code that kept this table up-to-date can also be removed (if any), as well as any related CSS stylesheet information that was specific to this table.
   
   It was a bit of oversight from my side, thanks for mentioning the clean up activity. I have removed the `refreshZKTable` function, calls to the function & also the 3 java programs under `/monitor/rest/zk` directory in the new commit. Hopefully, these are the one's (I think) that had kept the ZK table up-to-date. Let me know if there is anything else? 
   I also did a regression test and ensured the behaviour is still the same. 


-- 
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