kimsehwan96 commented on code in PR #824:
URL: https://github.com/apache/ranger/pull/824#discussion_r2736591920
##########
security-admin/src/main/java/org/apache/ranger/biz/XUserMgr.java:
##########
@@ -2964,6 +2964,9 @@ public int
createOrDeleteXGroupUserList(List<GroupUserInfo> groupUserInfoList) {
updatedGroups = groupUserInfoList.size();
Review Comment:
@mneethiraj Changed `xGroupUserService.createOrDeleteXGroupUsers()` method
to return boolean. If the method early returned with no actual updates then
return false and If actual update was performed then return true.
And increment `updatedGroups` and return its count as
`createOrDeleteGroupUserList` return and if at lease one update was really
performed then update user store version.
Alos I added test code about it.
--
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]
