DomGarguilo commented on code in PR #6366:
URL: https://github.com/apache/accumulo/pull/6366#discussion_r3236156953
##########
server/monitor/src/main/resources/org/apache/accumulo/monitor/resources/js/overview.js:
##########
@@ -35,7 +41,10 @@ $(function () {
* Makes the REST calls, generates the table with the new information
*/
function refreshOverview() {
- refreshDeploymentTables();
+ $.when(getInstanceInfo(), getDeployment()).then(function () {
+ refreshInstanceInfo();
+ refreshDeploymentTables();
+ });
Review Comment:
```suggestion
getInstanceInfo().then(function () {
refreshInstanceInfo();
});
getDeployment().then(function () {
refreshDeploymentTables();
});
```
I think we should separate these calls so if one fails it does not block the
other
--
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: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
