yandrey321 commented on code in PR #9427: URL: https://github.com/apache/ozone/pull/9427#discussion_r2603521638
########## hadoop-ozone/interface-storage/src/main/java/org/apache/hadoop/ozone/om/helpers/OmPrefixInfo.java: ########## @@ -18,24 +18,26 @@ package org.apache.hadoop.ozone.om.helpers; import com.google.common.base.Preconditions; -import java.util.ArrayList; -import java.util.LinkedList; +import com.google.common.collect.ImmutableList; +import java.util.LinkedHashMap; import java.util.List; import java.util.Map; import java.util.Objects; -import java.util.concurrent.CopyOnWriteArrayList; +import net.jcip.annotations.Immutable; import org.apache.hadoop.hdds.utils.db.Codec; import org.apache.hadoop.hdds.utils.db.CopyObject; import org.apache.hadoop.hdds.utils.db.DelegatedCodec; import org.apache.hadoop.hdds.utils.db.Proto2Codec; import org.apache.hadoop.ozone.OzoneAcl; +import org.apache.hadoop.ozone.OzoneConsts; +import org.apache.hadoop.ozone.audit.Auditable; import org.apache.hadoop.ozone.storage.proto.OzoneManagerStorageProtos.PersistedPrefixInfo; /** * Wrapper class for Ozone prefix path info, currently mainly target for ACL but * can be extended for other OzFS optimizations in future. */ -// TODO: support Auditable interface +@Immutable Review Comment: what is the reason for using this annotation? is it enforced by something? -- 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]
