dhavalshah9131 commented on code in PR #584: URL: https://github.com/apache/ranger/pull/584#discussion_r2226561957
########## security-admin/src/main/java/org/apache/ranger/rest/ServiceREST.java: ########## @@ -194,7 +191,8 @@ public class ServiceREST { public static final String PURGE_RECORD_TYPE_LOGIN_LOGS = "login_records"; public static final String PURGE_RECORD_TYPE_TRX_LOGS = "trx_records"; public static final String PURGE_RECORD_TYPE_POLICY_EXPORT_LOGS = "policy_export_logs"; - + private static final Logger LOG = LoggerFactory.getLogger(ServiceREST.class); Review Comment: Hi @mneethiraj , LOG seems to be private. As per steps mentioned [RANGER-5018](https://issues.apache.org/jira/browse/RANGER-5018) it gets shfited after public variables. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: dev-unsubscr...@ranger.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org