[
https://issues.apache.org/jira/browse/SENTRY-1512?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15621044#comment-15621044
]
Alexander Kolbasov commented on SENTRY-1512:
--------------------------------------------
[~colinma] I agree with you - we are actually opening an new persistence
manager for inner transaction, so this is, probably Ok. This means that
delegate.getRoleNamesForGroups(groups) is also fine since it will create
another instance to execute inner transaction.
> Refactor the database transaction management
> --------------------------------------------
>
> Key: SENTRY-1512
> URL: https://issues.apache.org/jira/browse/SENTRY-1512
> Project: Sentry
> Issue Type: Improvement
> Reporter: Colin Ma
> Assignee: Colin Ma
> Attachments: SENTRY-1512.001.patch
>
>
> The update for SENTRY-1422 refactor the database transaction management, this
> also should be ported to master.
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)
