ranger-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Velmurugan Periasamy <vperias...@hortonworks.com>
Subject Re: Review Request 71554: RANGER-2595: Improve policy delete performance using batch delete
Date Wed, 23 Oct 2019 14:06:48 GMT

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




security-admin/src/main/resources/META-INF/jpa_named_queries.xml
Lines 1601 (patched)
<https://reviews.apache.org/r/71554/#comment306037>

    Patch fails to apply. can you please check? Thanks.
    
    ```
    $ git apply --check -v ~/Downloads/0001-RANGER-2595-Improve-policy-delete-performance-using-.patch
    Checking patch security-admin/src/main/java/org/apache/ranger/biz/PolicyRefUpdater.java...
    Checking patch security-admin/src/main/java/org/apache/ranger/db/XXPolicyRefAccessTypeDao.java...
    Checking patch security-admin/src/main/java/org/apache/ranger/db/XXPolicyRefConditionDao.java...
    Checking patch security-admin/src/main/java/org/apache/ranger/db/XXPolicyRefDataMaskTypeDao.java...
    Checking patch security-admin/src/main/java/org/apache/ranger/db/XXPolicyRefGroupDao.java...
    Checking patch security-admin/src/main/java/org/apache/ranger/db/XXPolicyRefResourceDao.java...
    Checking patch security-admin/src/main/java/org/apache/ranger/db/XXPolicyRefRoleDao.java...
    error: while searching for:
            return ret;
        }
    
    }
    
    error: patch failed: security-admin/src/main/java/org/apache/ranger/db/XXPolicyRefRoleDao.java:96
    error: security-admin/src/main/java/org/apache/ranger/db/XXPolicyRefRoleDao.java: patch
does not apply
    Checking patch security-admin/src/main/java/org/apache/ranger/db/XXPolicyRefUserDao.java...
    Checking patch security-admin/src/main/resources/META-INF/jpa_named_queries.xml...
    Hunk #1 succeeded at 592 (offset 4 lines).
    Hunk #2 succeeded at 605 (offset 4 lines).
    Hunk #3 succeeded at 623 (offset 4 lines).
    Hunk #4 succeeded at 778 (offset 4 lines).
    Hunk #5 succeeded at 791 (offset 4 lines).
    Hunk #6 succeeded at 808 (offset 4 lines).
    error: while searching for:
    		</query>
    	</named-query>
    
    	<!-- XXTagChangeLog -->
    	<named-query name="XXTagChangeLog.findSinceVersion">
    
    error: patch failed: security-admin/src/main/resources/META-INF/jpa_named_queries.xml:1574
    error: security-admin/src/main/resources/META-INF/jpa_named_queries.xml: patch does not
apply
    ```


- Velmurugan Periasamy


On Oct. 17, 2019, 5:02 a.m., Andrew Luo wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/71554/
> -----------------------------------------------------------
> 
> (Updated Oct. 17, 2019, 5:02 a.m.)
> 
> 
> Review request for ranger.
> 
> 
> Bugs: RANGER-2595
>     https://issues.apache.org/jira/browse/RANGER-2595
> 
> 
> Repository: ranger
> 
> 
> Description
> -------
> 
> Policy deletion is slow for policies that have a large number of users or resources.
 This patch uses batch SQL deletion to speed up deletion.
> 
> 
> Diffs
> -----
> 
>   security-admin/src/main/java/org/apache/ranger/biz/PolicyRefUpdater.java 055cd382b

>   security-admin/src/main/java/org/apache/ranger/db/XXPolicyRefAccessTypeDao.java 1ef01bb28

>   security-admin/src/main/java/org/apache/ranger/db/XXPolicyRefConditionDao.java 2c04ab483

>   security-admin/src/main/java/org/apache/ranger/db/XXPolicyRefDataMaskTypeDao.java 258e3b0ba

>   security-admin/src/main/java/org/apache/ranger/db/XXPolicyRefGroupDao.java 08829d4f7

>   security-admin/src/main/java/org/apache/ranger/db/XXPolicyRefResourceDao.java e259ee864

>   security-admin/src/main/java/org/apache/ranger/db/XXPolicyRefRoleDao.java b92f8066f

>   security-admin/src/main/java/org/apache/ranger/db/XXPolicyRefUserDao.java f7b6131c6

>   security-admin/src/main/resources/META-INF/jpa_named_queries.xml 7e2139916 
> 
> 
> Diff: https://reviews.apache.org/r/71554/diff/2/
> 
> 
> Testing
> -------
> 
> Tested manually, ran build/tests
> 
> 
> Thanks,
> 
> Andrew Luo
> 
>


Mime
  • Unnamed multipart/alternative (inline, None, 0 bytes)
View raw message