----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/68871/ -----------------------------------------------------------
Review request for ranger, Ankita Sinha, Don Bosco Durai, Colm O hEigeartaigh, Gautam Borad, Madhan Neethiraj, Mehul Parikh, Nitin Galave, pengjianhua, Pradeep Agrawal, Ramesh Mani, Selvamohan Neethiraj, sam rome, Venkat Ranganathan, and Velmurugan Periasamy. Bugs: RANGER-2238 https://issues.apache.org/jira/browse/RANGER-2238 Repository: ranger Description ------- Here is related code: for(RangerPolicy.RangerPolicyItemCondition condition : policyItem.getConditions()) { if(condition.getType() == "ipaddress") { equals() should be used to compare Strings. Diffs ----- security-admin/src/main/java/org/apache/ranger/common/ServiceUtil.java 0292881b4 Diff: https://reviews.apache.org/r/68871/diff/1/ Testing ------- Thanks, Qiang Zhang