Skip to content

Commit 8a10d05

Browse files
committed
Fix small suggestion
1 parent 704a9ca commit 8a10d05

File tree

1 file changed

+3
-9
lines changed

1 file changed

+3
-9
lines changed

service/common/src/main/java/org/apache/polaris/service/admin/PolarisAdminService.java

Lines changed: 3 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -560,9 +560,7 @@ private void authorizeGrantOnPolicyOperationOrThrow(
560560
}
561561
}
562562

563-
PolarisResolvedPathWrapper policyWrapper =
564-
resolutionManifest.getResolvedPath(
565-
identifier, PolarisEntityType.POLICY, PolarisEntitySubType.ANY_SUBTYPE, true);
563+
PolarisResolvedPathWrapper policyWrapper = resolutionManifest.getResolvedPath(identifier, true);
566564
PolarisResolvedPathWrapper catalogRoleWrapper =
567565
resolutionManifest.getResolvedPath(catalogRoleName, true);
568566

@@ -2060,9 +2058,7 @@ private boolean grantPrivilegeOnPolicyEntityToRole(
20602058
findCatalogRoleByName(catalogName, catalogRoleName)
20612059
.orElseThrow(() -> new NotFoundException("CatalogRole %s not found", catalogRoleName));
20622060

2063-
PolarisResolvedPathWrapper resolvedPathWrapper =
2064-
resolutionManifest.getResolvedPath(
2065-
identifier, PolarisEntityType.POLICY, PolarisEntitySubType.ANY_SUBTYPE);
2061+
PolarisResolvedPathWrapper resolvedPathWrapper = resolutionManifest.getResolvedPath(identifier);
20662062
if (resolvedPathWrapper == null) {
20672063
throw new NoSuchPolicyException(String.format("Policy not exists: %s", identifier));
20682064
}
@@ -2092,9 +2088,7 @@ private boolean revokePrivilegeOnPolicyEntityFromRole(
20922088
findCatalogRoleByName(catalogName, catalogRoleName)
20932089
.orElseThrow(() -> new NotFoundException("CatalogRole %s not found", catalogRoleName));
20942090

2095-
PolarisResolvedPathWrapper resolvedPathWrapper =
2096-
resolutionManifest.getResolvedPath(
2097-
identifier, PolarisEntityType.POLICY, PolarisEntitySubType.ANY_SUBTYPE);
2091+
PolarisResolvedPathWrapper resolvedPathWrapper = resolutionManifest.getResolvedPath(identifier);
20982092
if (resolvedPathWrapper == null) {
20992093
throw new NoSuchPolicyException(String.format("Policy not exists: %s", identifier));
21002094
}

0 commit comments

Comments
 (0)