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:
[email protected]


Reply via email to