> On 十二月 1, 2022, 7:34 a.m., Madhan Neethiraj wrote:
> >

Thanks for your advice~
I have updated the code and Re-uploaded the patch file


- zhou


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/74224/#review224924
-----------------------------------------------------------


On 十二月 1, 2022, 6:47 a.m., zhou feng wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/74224/
> -----------------------------------------------------------
> 
> (Updated 十二月 1, 2022, 6:47 a.m.)
> 
> 
> Review request for ranger, Gautam Borad, Abhay Kulkarni, Madhan Neethiraj, 
> Nikhil P, Pradeep Agrawal, Ramesh Mani, Selvamohan Neethiraj, Sailaja 
> Polavarapu, and Velmurugan Periasamy.
> 
> 
> Bugs: RANGER-3950
>     https://issues.apache.org/jira/browse/RANGER-3950
> 
> 
> Repository: ranger
> 
> 
> Description
> -------
> 
> Fix the change of tag, tag-resource map , service resource is recorded to 
> x_tag_change_log
> https://issues.apache.org/jira/browse/RANGER-3950
> add the type of change of tag follow : 
>   TAG_RESOURCE_MAP_CREATE , TAG_RESOURCE_MAP_DELETE , SERVICE_RESOURCE_CREATE 
> , SERVICE_RESOURCE_DELETE
> 
> 
> Diffs
> -----
> 
>   
> agents-common/src/main/java/org/apache/ranger/plugin/model/RangerServiceResource.java
>  b7cf7cebb 
>   agents-common/src/main/java/org/apache/ranger/plugin/util/ServiceTags.java 
> 9b902789a 
>   security-admin/src/main/java/org/apache/ranger/biz/TagDBStore.java 
> e434cf1bb 
>   
> security-admin/src/main/java/org/apache/ranger/service/RangerServiceResourceServiceBase.java
>  329d3eeec 
> 
> 
> Diff: https://reviews.apache.org/r/74224/diff/1/
> 
> 
> Testing
> -------
> 
> Tested the below Rest API's to make sure everything works fine
> 1.POST   /service/tags/tags
> 2.PUT    /service/tag/guid/{guid}
> 3.DELETE  /service/tag/{id}
> 4.POST   /service/resources
> 5.PUT    /service/resource/{id}
> 6.DELETE  /service/resource/{id}
> 
> 
> Thanks,
> 
> zhou feng
> 
>

Reply via email to