-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/59511/
-----------------------------------------------------------
Review request for ranger, Alok Lal, Ankita Sinha, Don Bosco Durai, Colm O
hEigeartaigh, Gautam Borad, Madhan Neethiraj, Ramesh Mani, Selvamohan
Neethiraj, and Velmurugan Periasamy.
Bugs: RANGER-1610
https://issues.apache.org/jira/browse/RANGER-1610
Repository: ranger
Description
-------
Modify the Logger scope from Logger logger =
Logger.getLogger(XPolicyService.class);
to private static final Logger logger = Logger.getLogger(XPolicyService.class);
and keep the consistent info
if (xxGroup == null)
{ logger.error("No UserGroup found with this name : "+ group); throw
restErrorUtil.createRESTException( "No Group found with name : " + group,
MessageEnums.DATA_NOT_FOUND); }
to
if (xxGroup == null) {
logger.error("No Group found with name : "+ group);
throw restErrorUtil.createRESTException(
"No Group found with name : " + group, MessageEnums.DATA_NOT_FOUND);
Attachments
Diffs
-----
security-admin/src/main/java/org/apache/ranger/service/XPolicyService.java
066bc6a
Diff: https://reviews.apache.org/r/59511/diff/1/
Testing
-------
tested it
Thanks,
Qiang Zhang
