DomGarguilo commented on code in PR #6433:
URL: https://github.com/apache/accumulo/pull/6433#discussion_r3436448974
##########
server/monitor/src/main/java/org/apache/accumulo/monitor/next/InformationFetcher.java:
##########
Review Comment:
```suggestion
SystemInformation summary;
while ((summary = summaryRef.get()) == null) {
Thread.sleep(100);
}
return summary;
```
Not changed here but I noticed while reviewing this PR. In `getSummary()` we
call `summaryRef.get()` to check if its null then again to return its value.
This is less racy than calling `.get()` twice. Could include in this PR since
its related code.
--
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]
