neils-dev commented on code in PR #3781:
URL: https://github.com/apache/ozone/pull/3781#discussion_r998796755
##########
hadoop-hdds/server-scm/src/main/java/org/apache/hadoop/hdds/scm/node/DatanodeAdminMonitorImpl.java:
##########
@@ -82,6 +85,44 @@ public class DatanodeAdminMonitorImpl implements
DatanodeAdminMonitor {
private long unhealthyContainers = 0;
private long underReplicatedContainers = 0;
+ @SuppressFBWarnings(value = "SIC_INNER_SHOULD_BE_STATIC")
+ private final class ContainerStateInWorkflow {
Review Comment:
Thanks. Removed the suppress annotation and properly converted _instead_
to` static final nested class` _from_ the` final inner class.` As the inner
class does not refer to the outer class instance, it should indeed be a static
nested class.
--
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]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
