DomGarguilo commented on code in PR #2674:
URL: https://github.com/apache/accumulo/pull/2674#discussion_r864813357
##########
server/monitor/src/main/resources/org/apache/accumulo/monitor/resources/js/tservers.js:
##########
@@ -18,29 +18,25 @@
*/
"use strict";
-var tserversList;
+var tserversTable;
/**
* Creates tservers initial table
*/
$(document).ready(function() {
-
+
+ // hide the note about highlighted rows by default
+ $('#recovery-caption').hide();
Review Comment:
I'll try that out but I was under the impression that it needs to happen
here because this code will run again when the table is refreshed which will
clear the caption if needed. I think if it were in the FTL it might never be
hidden again once it is shown.
##########
server/monitor/src/main/resources/org/apache/accumulo/monitor/resources/js/tservers.js:
##########
@@ -92,10 +88,13 @@ $(document).ready(function() {
// reset background of each row
$(row).css('background-color', '');
- // return if current rows tserver is not recovering
+ // return if the current rows' tserver is not recovering
Review Comment:
Whoops, yea I think you are correct.
--
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: notifications-unsubscr...@accumulo.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
