diff --git a/oak-core/src/main/java/org/apache/jackrabbit/oak/security/user/GroupImpl.java b/oak-core/src/main/java/org/apache/jackrabbit/oak/security/user/GroupImpl.java index 473e03eb4c8..662398415de 100644 --- a/oak-core/src/main/java/org/apache/jackrabbit/oak/security/user/GroupImpl.java +++ b/oak-core/src/main/java/org/apache/jackrabbit/oak/security/user/GroupImpl.java @@ -18,7 +18,6 @@ import org.apache.jackrabbit.guava.common.base.Stopwatch; import org.apache.jackrabbit.guava.common.base.Strings; -import org.apache.jackrabbit.guava.common.collect.Maps; import org.apache.jackrabbit.api.security.user.Authorizable; import org.apache.jackrabbit.api.security.user.Group; import org.apache.jackrabbit.api.security.user.UserManager; @@ -298,7 +297,7 @@ private Set updateMembers(boolean isRemove, @NotNull String... memberIds } // calculate the contentID for each memberId and remember ids that cannot be processed - Map updateMap = Maps.newHashMapWithExpectedSize(memberIds.length); + Map updateMap = CollectionUtils.newHashMap(memberIds.length); MembershipProvider mp = getMembershipProvider(); for (String memberId : memberIds) { if (Strings.isNullOrEmpty(memberId)) { diff --git a/oak-core/src/main/java/org/apache/jackrabbit/oak/security/user/MembershipWriter.java b/oak-core/src/main/java/org/apache/jackrabbit/oak/security/user/MembershipWriter.java index 0f87fb3a977..107f824454c 100644 --- a/oak-core/src/main/java/org/apache/jackrabbit/oak/security/user/MembershipWriter.java +++ b/oak-core/src/main/java/org/apache/jackrabbit/oak/security/user/MembershipWriter.java @@ -16,13 +16,12 @@ */ package org.apache.jackrabbit.oak.security.user; +import java.util.HashMap; import java.util.HashSet; import java.util.Iterator; import java.util.Map; import java.util.Set; -import org.apache.jackrabbit.guava.common.collect.Maps; -import org.apache.jackrabbit.guava.common.collect.Sets; import org.apache.jackrabbit.JcrConstants; import org.apache.jackrabbit.oak.api.PropertyState; import org.apache.jackrabbit.oak.api.Tree; @@ -59,7 +58,7 @@ void setMembershipSizeThreshold(int membershipSizeThreshold) { * @return {@code true} if the member was added */ boolean addMember(@NotNull Tree groupTree, @NotNull String memberContentId) { - Map m = Maps.newHashMapWithExpectedSize(1); + Map m = new HashMap<>(1); m.put(memberContentId, "-"); return addMembers(groupTree, m).isEmpty(); } @@ -189,7 +188,7 @@ private static String nextRefNodeName(@NotNull Tree membersList) { * @return {@code true} if the member was removed. */ boolean removeMember(@NotNull Tree groupTree, @NotNull String memberContentId) { - Map m = Maps.newHashMapWithExpectedSize(1); + Map m = new HashMap<>(1); m.put(memberContentId, "-"); return removeMembers(groupTree, m).isEmpty(); } diff --git a/oak-core/src/main/java/org/apache/jackrabbit/oak/security/user/UserImporter.java b/oak-core/src/main/java/org/apache/jackrabbit/oak/security/user/UserImporter.java index c794337df52..06fa095b1b8 100644 --- a/oak-core/src/main/java/org/apache/jackrabbit/oak/security/user/UserImporter.java +++ b/oak-core/src/main/java/org/apache/jackrabbit/oak/security/user/UserImporter.java @@ -19,7 +19,6 @@ import org.apache.jackrabbit.guava.common.base.Stopwatch; import org.apache.jackrabbit.guava.common.collect.Iterables; import org.apache.jackrabbit.guava.common.collect.Maps; -import org.apache.jackrabbit.guava.common.collect.Sets; import org.apache.jackrabbit.api.JackrabbitSession; import org.apache.jackrabbit.api.security.principal.PrincipalIterator; import org.apache.jackrabbit.api.security.principal.PrincipalManager; @@ -33,6 +32,7 @@ import org.apache.jackrabbit.oak.api.Root; import org.apache.jackrabbit.oak.api.Tree; import org.apache.jackrabbit.oak.api.Type; +import org.apache.jackrabbit.oak.commons.collections.CollectionUtils; import org.apache.jackrabbit.oak.namepath.NamePathMapper; import org.apache.jackrabbit.oak.plugins.identifier.IdentifierManager; import org.apache.jackrabbit.oak.plugins.memory.PropertyStates; @@ -657,7 +657,7 @@ void process() throws RepositoryException { @NotNull Map getAuthorizablesToAdd(@NotNull Group gr, @NotNull Map toRemove, @NotNull Map nonExisting) throws RepositoryException { - Map toAdd = Maps.newHashMapWithExpectedSize(members.size()); + Map toAdd = CollectionUtils.newHashMap(members.size()); for (String contentId : members) { // NOTE: no need to check for re-mapped uuids with the referenceTracker because // ImportUUIDBehavior.IMPORT_UUID_CREATE_NEW is not supported for user/group imports (see line 189) diff --git a/oak-segment-tar/src/main/java/org/apache/jackrabbit/oak/segment/SegmentIdTable.java b/oak-segment-tar/src/main/java/org/apache/jackrabbit/oak/segment/SegmentIdTable.java index 0a07dfa582d..0ec87d1d507 100644 --- a/oak-segment-tar/src/main/java/org/apache/jackrabbit/oak/segment/SegmentIdTable.java +++ b/oak-segment-tar/src/main/java/org/apache/jackrabbit/oak/segment/SegmentIdTable.java @@ -19,7 +19,6 @@ package org.apache.jackrabbit.oak.segment; import static org.apache.jackrabbit.guava.common.collect.Lists.newArrayList; -import static org.apache.jackrabbit.guava.common.collect.Maps.newHashMapWithExpectedSize; import static java.util.Collections.nCopies; import java.lang.ref.WeakReference; @@ -30,6 +29,7 @@ import java.util.Set; import java.util.UUID; +import org.apache.jackrabbit.oak.commons.collections.CollectionUtils; import org.jetbrains.annotations.NotNull; import org.slf4j.Logger; import org.slf4j.LoggerFactory; @@ -130,8 +130,7 @@ void collectReferencedIds(Collection ids) { private synchronized Collection refresh() { int size = references.size(); - Map> ids = - newHashMapWithExpectedSize(size); + Map> ids = CollectionUtils.newHashMap(size); boolean hashCollisions = false; boolean emptyReferences = false; diff --git a/oak-segment-tar/src/main/java/org/apache/jackrabbit/oak/segment/file/tar/GraphLoader.java b/oak-segment-tar/src/main/java/org/apache/jackrabbit/oak/segment/file/tar/GraphLoader.java index 6837ffe5833..0dfd076d6ac 100644 --- a/oak-segment-tar/src/main/java/org/apache/jackrabbit/oak/segment/file/tar/GraphLoader.java +++ b/oak-segment-tar/src/main/java/org/apache/jackrabbit/oak/segment/file/tar/GraphLoader.java @@ -18,7 +18,6 @@ */ package org.apache.jackrabbit.oak.segment.file.tar; -import static org.apache.jackrabbit.guava.common.collect.Maps.newHashMapWithExpectedSize; import static org.apache.jackrabbit.oak.segment.file.tar.TarConstants.GRAPH_MAGIC; import java.io.IOException; @@ -29,6 +28,7 @@ import java.util.zip.CRC32; import org.apache.jackrabbit.oak.commons.Buffer; +import org.apache.jackrabbit.oak.commons.collections.CollectionUtils; import org.apache.jackrabbit.oak.segment.util.ReaderAtEnd; import org.slf4j.Logger; import org.slf4j.LoggerFactory; @@ -93,7 +93,7 @@ public static Buffer loadGraph(ReaderAtEnd readerAtEnd) throws IOException { public static Map> parseGraph(Buffer buffer) { int nEntries = buffer.getInt(buffer.limit() - 12); - Map> graph = newHashMapWithExpectedSize(nEntries); + Map> graph = CollectionUtils.newHashMap(nEntries); for (int i = 0; i < nEntries; i++) { long msb = buffer.getLong();