ctubbsii commented on code in PR #5873:
URL: https://github.com/apache/accumulo/pull/5873#discussion_r2334459375
##########
server/monitor/src/main/java/org/apache/accumulo/monitor/next/InformationFetcher.java:
##########
@@ -236,9 +236,9 @@ public void newConnectionEvent() {
}
// Protect against NPE and wait for initial data gathering
- public SystemInformation getSummary() {
+ public SystemInformation getSummary() throws InterruptedException {
while (summaryRef.get() == null) {
- Thread.onSpinWait();
+ Thread.sleep(100);
Review Comment:
It seems like it would be better to handle InterruptedException here, rather
than throw it out and handle it everywhere else.
--
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]
