review: add TODO in listPolicies

XN137 · XN137 · commit 3cc904d7c3b2 · 2025-08-19T17:45:53.000+02:00
diff --git a/runtime/service/src/main/java/org/apache/polaris/service/catalog/policy/PolicyCatalog.java b/runtime/service/src/main/java/org/apache/polaris/service/catalog/policy/PolicyCatalog.java
@@ -161,6 +161,7 @@ public List<PolicyIdentifier> listPolicies(Namespace namespace, PolicyType polic
     }
 
     List<PolarisEntity> catalogPath = resolvedEntities.getRawFullPath();
+    // TODO: when the "policyType" filter is null we should only call "listEntities" instead
     return metaStoreManager
         .loadEntitiesAll(
             callContext.getPolarisCallContext(),

Original file line number	Diff line number	Diff line change
`@@ -161,6 +161,7 @@ public List<PolicyIdentifier> listPolicies(Namespace namespace, PolicyType polic`
`161`	`161`	`}`
`162`	`162`
`163`	`163`	`List<PolarisEntity> catalogPath = resolvedEntities.getRawFullPath();`
	`164`	`+ // TODO: when the "policyType" filter is null we should only call "listEntities" instead`
`164`	`165`	`return metaStoreManager`
`165`	`166`	`.loadEntitiesAll(`
`166`	`167`	`callContext.getPolarisCallContext(),`