-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/67782/
-----------------------------------------------------------
Review request for ranger.
Bugs: RANGER-2144
https://issues.apache.org/jira/browse/RANGER-2144
Repository: ranger
Description
-------
Currently, RangerDaoManagerBase is used to construct new DAO instances, instead
of relying on Spring to provide one for it. This would reduce the code to write
and run, and make it less interdependent, and simplifies the tests, as less
code would be needed to mock.
As RangerDaoManagerBase is used everywhere, and to avoid having huge patches,
it would be better do it in smaller steps.
Diffs
-----
security-admin/src/main/java/org/apache/ranger/biz/AssetMgr.java 941691aaa
security-admin/src/main/java/org/apache/ranger/biz/RangerBizUtil.java
b304e3e99
security-admin/src/main/java/org/apache/ranger/biz/RangerPolicyRetriever.java
463957c49
security-admin/src/main/java/org/apache/ranger/biz/RangerTagDBRetriever.java
55505723d
security-admin/src/main/java/org/apache/ranger/biz/ServiceDBStore.java
f00d31185
security-admin/src/main/java/org/apache/ranger/biz/SessionMgr.java 4a426b78a
security-admin/src/main/java/org/apache/ranger/biz/TagDBStore.java 8341a7354
security-admin/src/main/java/org/apache/ranger/biz/UserMgr.java 6d94e4f61
security-admin/src/main/java/org/apache/ranger/biz/XUserMgr.java 7f5eab7eb
security-admin/src/main/java/org/apache/ranger/common/StringUtil.java
82afa2708
security-admin/src/main/java/org/apache/ranger/db/RangerDaoManager.java
2788a6109
security-admin/src/main/java/org/apache/ranger/db/RangerDaoManagerBase.java
da89e041c
security-admin/src/main/java/org/apache/ranger/db/XXAssetDao.java 1cd83badc
security-admin/src/main/java/org/apache/ranger/db/XXPortalUserDao.java
d383cf596
security-admin/src/main/java/org/apache/ranger/db/XXResourceDao.java
24a0e63ef
security-admin/src/main/java/org/apache/ranger/db/XXUserDao.java 58bf4d891
security-admin/src/main/java/org/apache/ranger/patch/PatchMigration_J10002.java
0af7a1da2
security-admin/src/main/java/org/apache/ranger/patch/PatchTagModulePermission_J10005.java
6f9350da2
security-admin/src/main/java/org/apache/ranger/rest/PublicAPIs.java 7df957973
security-admin/src/main/java/org/apache/ranger/service/RangerPluginInfoService.java
2bbf0d413
security-admin/src/main/java/org/apache/ranger/service/RangerPolicyService.java
519d8e94d
security-admin/src/main/java/org/apache/ranger/service/RangerPolicyWithAssignedIdService.java
540a29eb2
security-admin/src/main/java/org/apache/ranger/service/RangerServiceDefServiceBase.java
5cbe47ad3
security-admin/src/main/java/org/apache/ranger/service/RangerServiceResourceService.java
d7256802a
security-admin/src/main/java/org/apache/ranger/service/RangerTagDefService.java
82eb252e6
security-admin/src/main/java/org/apache/ranger/service/RangerTagResourceMapService.java
89c451ee2
security-admin/src/main/java/org/apache/ranger/service/RangerTagService.java
28b9115fa
security-admin/src/main/java/org/apache/ranger/service/RangerTagServiceBase.java
2e2c04f47
security-admin/src/main/java/org/apache/ranger/service/UserService.java
9ca6a4fb3
security-admin/src/main/java/org/apache/ranger/service/XModuleDefService.java
d5ca38548
security-admin/src/main/java/org/apache/ranger/service/XPolicyService.java
066bc6ae7
security-admin/src/main/java/org/apache/ranger/service/XPortalUserService.java
c96878d21
security-admin/src/main/java/org/apache/ranger/service/XResourceService.java
b3e7bd7d7
security-admin/src/main/java/org/apache/ranger/service/XUgsyncAuditInfoService.java
7f3d0c70d
security-admin/src/main/java/org/apache/ranger/service/XUserPermissionServiceBase.java
bc6c21eae
security-admin/src/test/java/org/apache/ranger/biz/TestRangerBizUtil.java
22e290a66
security-admin/src/test/java/org/apache/ranger/biz/TestServiceDBStore.java
b0b011022
security-admin/src/test/java/org/apache/ranger/biz/TestUserMgr.java 10453a517
security-admin/src/test/java/org/apache/ranger/biz/TestXUserMgr.java
39c60a946
security-admin/src/test/java/org/apache/ranger/rest/TestPublicAPIs.java
2bf5ee6c9
security-admin/src/test/java/org/apache/ranger/rest/TestUserREST.java
3c3051b39
security-admin/src/test/java/org/apache/ranger/service/TestRangerServiceDefService.java
a36b19879
security-admin/src/test/java/org/apache/ranger/service/TestRangerServiceDefServiceBase.java
74f1dd0c4
security-admin/src/test/java/org/apache/ranger/service/TestRangerTagDefService.java
8f19ffd78
security-admin/src/test/java/org/apache/ranger/service/TestRangerTagResourceMapService.java
27ec8e140
security-admin/src/test/java/org/apache/ranger/service/TestRangerTagService.java
1175989e2
security-admin/src/test/java/org/apache/ranger/service/TestUserService.java
c48739b94
Diff: https://reviews.apache.org/r/67782/diff/1/
Testing
-------
Tested with unit tests and on a live cluster.
Thanks,
Zsombor Gegesy
