Capping heap size used by StatefulIndexPrivileges

Signed-off-by: Nils Bandener <[email protected]>
nibix · Aug 2, 2024 · 0252f5b · 0252f5b
1 parent 78db68b
commit 0252f5b
Showing 1 changed file with 2 additions and 5 deletions.
diff --git a/src/main/java/org/opensearch/security/privileges/ActionPrivileges.java b/src/main/java/org/opensearch/security/privileges/ActionPrivileges.java
@@ -868,7 +868,7 @@ static class StatefulIndexPrivileges {
                 roles.getCEntries().keySet()
             );
             CompactMapGroupBuilder<String, DeduplicatingCompactSubSetBuilder.SubSetBuilder<String>> indexMapBuilder =
-                    new CompactMapGroupBuilder<>(indices.keySet(), (k2) -> roleSetBuilder.createSubSetBuilder());
+                new CompactMapGroupBuilder<>(indices.keySet(), (k2) -> roleSetBuilder.createSubSetBuilder());
 
             top: for (Map.Entry<String, RoleV7> entry : roles.getCEntries().entrySet()) {
                 try {
@@ -905,10 +905,7 @@ static class StatefulIndexPrivileges {
                                     CompactMapGroupBuilder.MapBuilder<
                                         String,
                                         DeduplicatingCompactSubSetBuilder.SubSetBuilder<String>> indexToRoles = actionToIndexToRoles
-                                            .computeIfAbsent(
-                                                action,
-                                                k -> indexMapBuilder.createMapBuilder()
-                                            );
+                                            .computeIfAbsent(action, k -> indexMapBuilder.createMapBuilder());
 
                                     indexToRoles.get(indicesEntry.getKey()).add(roleName);