You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@accumulo.apache.org by mm...@apache.org on 2020/10/05 14:57:33 UTC

[accumulo] branch main updated: Fix js refresh functions for Monitor problems tables

This is an automated email from the ASF dual-hosted git repository.

mmiller pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/accumulo.git


The following commit(s) were added to refs/heads/main by this push:
     new 8789412  Fix js refresh functions for Monitor problems tables
8789412 is described below

commit 8789412f2fa1fb7f3eced2789d9bc8abe066fbb8
Author: Mike Miller <mm...@apache.org>
AuthorDate: Mon Oct 5 10:56:34 2020 -0400

    Fix js refresh functions for Monitor problems tables
---
 .../org/apache/accumulo/monitor/resources/js/functions.js  | 10 ++++++++--
 .../org/apache/accumulo/monitor/resources/js/problems.js   | 14 +++-----------
 2 files changed, 11 insertions(+), 13 deletions(-)

diff --git a/server/monitor/src/main/resources/org/apache/accumulo/monitor/resources/js/functions.js b/server/monitor/src/main/resources/org/apache/accumulo/monitor/resources/js/functions.js
index 66a2b80..21a910d 100644
--- a/server/monitor/src/main/resources/org/apache/accumulo/monitor/resources/js/functions.js
+++ b/server/monitor/src/main/resources/org/apache/accumulo/monitor/resources/js/functions.js
@@ -475,7 +475,10 @@ function clearTableProblems(tableID) {
   var call = '/rest/problems/summary?s=' + tableID;
   // Change plus sign to use ASCII value to send it as a URL query parameter
   call = sanitize(call);
-  $.post(call);
+  // make the rest call, passing success function callback
+  $.post(call, function () {
+    refreshProblems();
+  });
 }
 
 /**
@@ -490,7 +493,10 @@ function clearDetailsProblems(table, resource, type) {
    resource + '&ptype=' + type;
   // Changes plus sign to use ASCII value to send it as a URL query parameter
   call = sanitize(call);
-  $.post(call);
+  // make the rest call, passing success function callback
+  $.post(call, function () {
+    refreshProblems();
+  });
 }
 
 /**
diff --git a/server/monitor/src/main/resources/org/apache/accumulo/monitor/resources/js/problems.js b/server/monitor/src/main/resources/org/apache/accumulo/monitor/resources/js/problems.js
index 4301fde..aee3083 100644
--- a/server/monitor/src/main/resources/org/apache/accumulo/monitor/resources/js/problems.js
+++ b/server/monitor/src/main/resources/org/apache/accumulo/monitor/resources/js/problems.js
@@ -97,19 +97,15 @@ $(document).ready(function() {
       }
     ]
   });
-  refreshProblemSummaryTable();
 });
 
 /**
  * Makes the REST calls, generates the tables with the new information
  */
 function refreshProblems() {
-  getProblemSummary().then(function() {
-    refreshProblemSummaryTable();
-  });
-  getProblemDetails().then(function() {
-    refreshProblemDetailsTable();
-  });
+  refreshNavBar();
+  refreshProblemSummaryTable();
+  refreshProblemDetailsTable();
 }
 
 /**
@@ -126,8 +122,6 @@ function refresh() {
  */
 function clearTableProblemsTable(tableID) {
   clearTableProblems(tableID);
-  refreshProblemSummaryTable();
-  refreshNavBar();
 }
 
 /**
@@ -139,8 +133,6 @@ function clearTableProblemsTable(tableID) {
  */
 function clearDetailsProblemsTable(table, resource, type) {
   clearDetailsProblems(table, resource, type);
-  refreshProblemDetailsTable();
-  refreshNavBar();
 }
 
 /**