vyommani opened a new pull request, #1141:
URL: https://github.com/apache/ranger/pull/1141

   …uses CME/NPE for delegated-admin User
   
   ## What changes were proposed in this pull request?
   
   this patch is built on top of Madhan's approach.
   
   normalize() is correctly protected in the base patch, but 
ServiceDefUtil.normalizeAccessTypeDefs() — called separately right after it in 
RangerPolicyRepository and PolicyEngine — was still mutating the shared 
serviceDef's access-type lists with no synchronization. This PR moves that 
logic into RangerServiceDef with the same double-checked-locking pattern 
already used for normalize(), and updates both call sites.
   
   
   ## How was this patch tested?
   
   Includes a concurrency test that fails against the base patch alone, and 
passes once this fix is applied on top.
   


-- 
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]

Reply via email to