-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/65485/
-----------------------------------------------------------
Review request for ranger.
Bugs: RANGER-1971
https://issues.apache.org/jira/browse/RANGER-1971
Repository: ranger
Description
-------
We should switch to use for-each loops in the code, as it is more readable and
concise than using an index.
Diffs
-----
agents-audit/src/main/java/org/apache/ranger/audit/provider/hdfs/HdfsLogDestination.java
065e8b07
agents-audit/src/main/java/org/apache/ranger/audit/queue/AuditFileSpool.java
9abd99f5
agents-common/src/main/java/org/apache/ranger/plugin/geo/GeolocationMetadata.java
d27a0308
agents-common/src/main/java/org/apache/ranger/plugin/geo/RangerGeolocationData.java
99d6027f
agents-common/src/test/java/org/apache/ranger/plugin/policyengine/TestCacheMap.java
5f39b224
credentialbuilder/src/main/java/org/apache/ranger/credentialapi/buildks.java
b8bdb6fa
hbase-agent/src/main/java/org/apache/ranger/authorization/hbase/RangerAuthorizationCoprocessor.java
e30f7957
hive-agent/src/main/java/org/apache/ranger/authorization/hive/authorizer/RangerHiveAuthorizer.java
fa84b138
jisql/src/main/java/org/apache/util/sql/Jisql.java 53a6ca4f
kms/src/main/java/org/apache/hadoop/crypto/key/kms/server/KMSMetricUtil.java
1527681a
plugin-solr/src/main/java/org/apache/ranger/services/solr/client/ServiceSolrClient.java
5875a298
security-admin/src/main/java/org/apache/ranger/biz/AssetMgr.java a53d46af
security-admin/src/main/java/org/apache/ranger/biz/UserMgr.java 272dec40
security-admin/src/main/java/org/apache/ranger/common/RESTErrorUtil.java
967804ad
security-admin/src/main/java/org/apache/ranger/common/StringUtil.java
045e07c4
security-admin/src/main/java/org/apache/ranger/patch/cliutil/MetricUtil.java
d1ab0d09
security-admin/src/main/java/org/apache/ranger/service/UserService.java
3fb279e9
security-admin/src/main/java/org/apache/ranger/service/filter/RangerRESTAPIFilter.java
551f824b
security-admin/src/main/java/org/apache/ranger/util/RangerRestUtil.java
fe7a53e5
ugsync/src/main/java/org/apache/ranger/ldapusersync/process/LdapUserGroupBuilder.java
8dc147ca
ugsync/src/main/java/org/apache/ranger/unixusersync/process/PolicyMgrUserGroupBuilder.java
ade2ee71
Diff: https://reviews.apache.org/r/65485/diff/1/
Testing
-------
Thanks,
Colm O hEigeartaigh