----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/74634/#review225806 -----------------------------------------------------------
agents-common/src/main/java/org/apache/ranger/plugin/model/RangerGds.java Lines 480 (patched) <https://reviews.apache.org/r/74634/#comment314245> Consider using GdsPermission for permissionForCaller, instead of String. agents-common/src/main/java/org/apache/ranger/plugin/model/RangerGds.java Lines 540 (patched) <https://reviews.apache.org/r/74634/#comment314241> Field id is present in base class RangerBaseModelObject. Please review and remove from DataShareInDatasetSummary. agents-common/src/main/java/org/apache/ranger/plugin/model/RangerGds.java Lines 547 (patched) <https://reviews.apache.org/r/74634/#comment314242> Consider using GdsShareStatus for shareStatus, instead of String. security-admin/src/main/java/org/apache/ranger/biz/GdsDBStore.java Line 135 (original), 143 (patched) <https://reviews.apache.org/r/74634/#comment314244> getDatasetHeaders() => getDatasetSummary() security-admin/src/main/java/org/apache/ranger/service/RangerGdsSharedResourceService.java Line 251 (original), 251 (patched) <https://reviews.apache.org/r/74634/#comment314243> resource-count in a dataShare shouldn't vary across datasets it is associated to. If yes, does this method need datasetId parameter? - Madhan Neethiraj On Oct. 2, 2023, 2:51 p.m., Subhrat Chaudhary wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/74634/ > ----------------------------------------------------------- > > (Updated Oct. 2, 2023, 2:51 p.m.) > > > Review request for ranger and Madhan Neethiraj. > > > Bugs: https://issues.apache.org/jira/browse/RANGER-4446 > > https://issues.apache.org/jira/browse/https://issues.apache.org/jira/browse/RANGER-4446 > > > Repository: ranger > > > Description > ------- > > We need additional details in the response for DatasetHeaderInfo like summary > for DataShare included int he dataset. > > > Diffs > ----- > > agents-common/src/main/java/org/apache/ranger/plugin/model/RangerGds.java > a1c883356 > security-admin/src/main/java/org/apache/ranger/biz/GdsDBStore.java > ce4769569 > security-admin/src/main/java/org/apache/ranger/rest/GdsREST.java 56b908625 > > security-admin/src/main/java/org/apache/ranger/security/context/RangerAPIList.java > b22208773 > > security-admin/src/main/java/org/apache/ranger/service/RangerGdsSharedResourceService.java > eadbb9228 > > > Diff: https://reviews.apache.org/r/74634/diff/1/ > > > Testing > ------- > > Updated dataset summary details: > { > "id": 1, > "guid": "35081b85-7c1e-48b7-aca4-f7c7b9aa7b18", > "isEnabled": true, > "createdBy": "Admin", > "updatedBy": "Admin", > "createTime": 1696075254523, > "updateTime": 1696075254534, > "version": 1, > "name": "DS1", > "principalsCountByType": { > "USER": 2, > "GROUP": 0, > "ROLE": 0 > }, > "projectsCount": 0, > "permissionForCaller": "ADMIN", > "totalResourceCount": 2, > "dataShares": [ > { > "id": 3, > "guid": "b2c938ac-8da7-4861-a29a-cfed5bdb62c8", > "isEnabled": true, > "createdBy": "Admin", > "updatedBy": "Admin", > "createTime": 1696073077134, > "updateTime": 1696073077136, > "version": 1, > "name": "datashare3", > "serviceId": 1, > "serviceName": "hive1", > "zoneId": 2, > "zoneName": "zone1", > "resourceCount": 1, > "shareStatus": "REQUESTED" > }, > { > "id": 4, > "guid": "bf92da3b-3a4b-445f-8fa0-11f5a02306f3", > "isEnabled": true, > "createdBy": "Admin", > "updatedBy": "Admin", > "createTime": 1696146862432, > "updateTime": 1696146862447, > "version": 1, > "name": "datashare4", > "serviceId": 1, > "serviceName": "hive1", > "zoneId": 2, > "zoneName": "zone1", > "resourceCount": 1, > "shareStatus": "REQUESTED" > } > ] > } > > > Thanks, > > Subhrat Chaudhary > >
