[ 
https://issues.apache.org/jira/browse/SENTRY-1356?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15349654#comment-15349654
 ] 

Sravya Tirukkovalur edited comment on SENTRY-1356 at 6/25/16 2:08 PM:
----------------------------------------------------------------------

So following two things need evaluation to understand if there any problems 
currently:
1. Is each event triggering a new PathsUpdate?
2. What are the events which have more than one pathChanges? Is this behavior 
acceptable?


was (Author: sravya):
So following three things need evaluation to understand if there any problems 
currently:
1. Is each event triggering a new PathsUpdate?
2. What are the events which have more than one pathChanges? Is this behavior 
acceptable?

> Evaluate the current approach of updates from HMS
> -------------------------------------------------
>
>                 Key: SENTRY-1356
>                 URL: https://issues.apache.org/jira/browse/SENTRY-1356
>             Project: Sentry
>          Issue Type: Sub-task
>            Reporter: Sravya Tirukkovalur
>
> Would be good to reevaluate the current approach of HMS updates. Currently, 
> update contains a list of addPaths and delPaths. Some concerns we need to 
> evaluate:
> 1. Hard to capture order in this model.
> 2. In UpdateatableAuthzPaths.applyPartialUpdate we seem to be adding the 
> newPaths first and removing the delPaths. One caveat here is: if the newPath 
> and oldPath are same, we end up deleting the path. See SENTRY-1346 for one 
> such example.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

Reply via email to