-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/56890/
-----------------------------------------------------------

Review request for ranger.


Bugs: RANGER-1377
    https://issues.apache.org/jira/browse/RANGER-1377


Repository: ranger


Description
-------

Calling toString on a String is meaningless, or when it's already added to a 
string. 
Concatenating strings inside a String.append is wasteful, as "" + obj instead 
of String.valueOf(obj)


Diffs
-----

  security-admin/src/main/java/org/apache/ranger/biz/RangerBizUtil.java 5521ae8 
  security-admin/src/main/java/org/apache/ranger/biz/ServiceDBStore.java 
15f205a 
  security-admin/src/main/java/org/apache/ranger/biz/ServiceMgr.java f512049 
  security-admin/src/main/java/org/apache/ranger/common/DateUtil.java 7babd80 
  security-admin/src/main/java/org/apache/ranger/common/PropertiesUtil.java 
22d4b89 
  security-admin/src/main/java/org/apache/ranger/common/RangerSearchUtil.java 
4fb52a4 
  security-admin/src/main/java/org/apache/ranger/common/SearchGroup.java 
77119b1 
  security-admin/src/main/java/org/apache/ranger/common/SearchUtil.java f4fcfb2 
  security-admin/src/main/java/org/apache/ranger/common/StringUtil.java da7231e 
  security-admin/src/main/java/org/apache/ranger/db/XXResourceDao.java 9fa77f2 
  security-admin/src/main/java/org/apache/ranger/entity/XXServiceResource.java 
961627a 
  
security-admin/src/main/java/org/apache/ranger/entity/XXServiceResourceElement.java
 16dca97 
  
security-admin/src/main/java/org/apache/ranger/entity/XXServiceResourceElementValue.java
 82ce8a0 
  security-admin/src/main/java/org/apache/ranger/entity/XXTag.java 9155385 
  security-admin/src/main/java/org/apache/ranger/entity/XXTagAttribute.java 
1c88288 
  security-admin/src/main/java/org/apache/ranger/entity/XXTagAttributeDef.java 
5c6ff58 
  security-admin/src/main/java/org/apache/ranger/entity/XXTagDef.java 818908b 
  security-admin/src/main/java/org/apache/ranger/entity/XXTagResourceMap.java 
4b8b3ec 
  
security-admin/src/main/java/org/apache/ranger/patch/PatchTagModulePermission_J10005.java
 6f9350d 
  
security-admin/src/main/java/org/apache/ranger/patch/cliutil/DbToSolrMigrationUtil.java
 1ddd367 
  security-admin/src/main/java/org/apache/ranger/rest/PublicAPIs.java d3c22d7 
  
security-admin/src/main/java/org/apache/ranger/security/handler/RangerAuthenticationProvider.java
 3fdd5c2 
  
security-admin/src/main/java/org/apache/ranger/security/web/filter/RangerKRBAuthenticationFilter.java
 c8d22aa 
  
security-admin/src/main/java/org/apache/ranger/security/web/filter/RangerSSOAuthenticationFilter.java
 7d8fdb2 
  
security-admin/src/main/java/org/apache/ranger/security/web/filter/RangerUsernamePasswordAuthenticationFilter.java
 62e23c0 
  
security-admin/src/main/java/org/apache/ranger/service/AbstractBaseResourceService.java
 9a4aa3b 
  
security-admin/src/main/java/org/apache/ranger/service/RangerBaseModelService.java
 482f10c 
  
security-admin/src/main/java/org/apache/ranger/service/RangerPolicyService.java 
8d9776a 
  
security-admin/src/main/java/org/apache/ranger/service/RangerServiceService.java
 004524b 
  security-admin/src/main/java/org/apache/ranger/service/XAssetService.java 
81c720d 
  security-admin/src/main/java/org/apache/ranger/service/XAuditMapService.java 
1dfe19f 
  security-admin/src/main/java/org/apache/ranger/service/XGroupService.java 
c478dbc 
  security-admin/src/main/java/org/apache/ranger/service/XGroupUserService.java 
0d735da 
  security-admin/src/main/java/org/apache/ranger/service/XPermMapService.java 
bc335cc 
  security-admin/src/main/java/org/apache/ranger/service/XPolicyService.java 
f3f7c80 
  
security-admin/src/main/java/org/apache/ranger/service/XPortalUserService.java 
bb12e3e 
  security-admin/src/main/java/org/apache/ranger/service/XResourceService.java 
817fdda 
  security-admin/src/main/java/org/apache/ranger/service/XUserService.java 
0f8156a 
  
security-admin/src/main/java/org/apache/ranger/solr/SolrAccessAuditsService.java
 917c760 
  security-admin/src/main/java/org/apache/ranger/solr/SolrUtil.java a851c3e 
  security-admin/src/main/java/org/apache/ranger/view/VXKmsKey.java d7a0b4d 
  security-admin/src/main/java/org/apache/ranger/view/VXMetricPolicyCount.java 
bb18775 
  security-admin/src/test/java/org/apache/ranger/audit/TestAuditQueue.java 
dee3156 

Diff: https://reviews.apache.org/r/56890/diff/


Testing
-------


Thanks,

Zsombor Gegesy

Reply via email to