avijayanhwx commented on a change in pull request #2759:
URL: https://github.com/apache/ozone/pull/2759#discussion_r742987957
##########
File path:
hadoop-ozone/ozone-manager/src/main/java/org/apache/hadoop/ozone/om/OMMultiTenantManagerImpl.java
##########
@@ -427,16 +360,53 @@ public void deactivateUser(String accessID)
}
@Override
- public List<String> listAllAccessIDs(String tenantID)
+ public boolean isTenantAdmin(String user, String tenantName) {
+ return true;
+ }
+
+ @Override
+ public TenantUserList listUsersInTenant(String tenantID, String prefix)
throws IOException {
- return null;
+
+ if (!omMetadataManager.getTenantStateTable().isExist(tenantID)) {
+ throw new IOException("Tenant '" + tenantID + "' not found!");
+ }
+
+ List<TenantUserAccessId> userAccessIds = new ArrayList<>();
+ CachedTenantInfo cachedTenantInfo = tenantCache.get(tenantID);
+ if (cachedTenantInfo == null) {
+ throw new IOException("Inconsistent in memory Tenant cache '" + tenantID
+ + "' not found in cache, but present in OM DB!");
+ }
+
+ cachedTenantInfo.getTenantUsers().stream()
+ .filter(
+ k -> StringUtils.isEmpty(prefix) || k.getKey().startsWith(prefix))
+ .forEach(
+ k -> userAccessIds.add(
+ TenantUserAccessId.newBuilder()
+ .setUser(k.getKey())
+ .setAccessId(k.getValue())
+ .build()));
+ return new TenantUserList(tenantID, userAccessIds);
Review comment:
Yes, I have created an internal tracking task for that. We can handle it
in a follow up task.
--
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]
