[airavata-custos] branch develop updated: resolve performance issue: Sharing service userHasAccess method

[isjarana]

This is an automated email from the ASF dual-hosted git repository.

isjarana pushed a commit to branch develop
in repository https://gitbox.apache.org/repos/asf/airavata-custos.git


The following commit(s) were added to refs/heads/develop by this push:
     new 0639900  resolve performance issue: Sharing service userHasAccess 
method
     new cf1f3ea  Merge pull request #128 from isururanawaka/pythonSDK
0639900 is described below

commit 0639900389bca5e31c1b25f0e66ecfb8bfccc9c8
Author: Isuru Ranawaka <irjan...@gmail.com>
AuthorDate: Tue Dec 15 10:09:46 2020 -0500

    resolve performance issue: Sharing service userHasAccess method
---
 .../sharing/management/service/SharingManagementService.java   | 10 ++++------
 1 file changed, 4 insertions(+), 6 deletions(-)

diff --git 
a/custos-integration-services/sharing-management-service-parent/sharing-management-service/src/main/java/org/apache/custos/sharing/management/service/SharingManagementService.java
 
b/custos-integration-services/sharing-management-service-parent/sharing-management-service/src/main/java/org/apache/custos/sharing/management/service/SharingManagementService.java
index 9c415ea..5a9f498 100644
--- 
a/custos-integration-services/sharing-management-service-parent/sharing-management-service/src/main/java/org/apache/custos/sharing/management/service/SharingManagementService.java
+++ 
b/custos-integration-services/sharing-management-service-parent/sharing-management-service/src/main/java/org/apache/custos/sharing/management/service/SharingManagementService.java
@@ -581,14 +581,8 @@ public class SharingManagementService extends 
SharingManagementServiceImplBase {
             long tenantId = request.getTenantId();
 
 
-            for (String username : request.getOwnerIdList()) {
-
-                validateAndGetUserProfile(username, clientId, clientSec, 
tenantId);
-            }
-
             UserProfile profile = 
UserProfile.newBuilder().setUsername(request.getOwnerId(0)).build();
 
-
             UserProfileRequest userProfileRequest = 
UserProfileRequest.newBuilder()
                     .setTenantId(tenantId)
                     .setProfile(profile)
@@ -609,6 +603,10 @@ public class SharingManagementService extends 
SharingManagementServiceImplBase {
             responseObserver.onNext(status);
             responseObserver.onCompleted();
 
+            for (String username : request.getOwnerIdList()) {
+
+                validateAndGetUserProfile(username, clientId, clientSec, 
tenantId);
+            }
 
         } catch (Exception ex) {
             String msg = "Error occurred at userHasAccess " + ex.getMessage();