----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50591/ -----------------------------------------------------------
Review request for ranger, Madhan Neethiraj and Velmurugan Periasamy. Repository: ranger Description ------- RANGER-1124:Code changes to avoid potential NPEs and good coding practices in hbase-agent, hdfs-agent, hive-agent, knox-agent, plugin-kafka, plugin-nifi, plugin-solr, ranger-plugin-classloader, agents-audit - medium risk Diffs ----- agents-audit/src/main/java/org/apache/ranger/audit/destination/FileAuditDestination.java c6cd8b2 agents-audit/src/main/java/org/apache/ranger/audit/destination/HDFSAuditDestination.java 9da97ed agents-audit/src/main/java/org/apache/ranger/audit/destination/SolrAuditDestination.java 46f33a5 agents-audit/src/main/java/org/apache/ranger/audit/provider/AuditProviderFactory.java e3e818c agents-audit/src/main/java/org/apache/ranger/audit/provider/LocalFileLogBuffer.java a671165 agents-audit/src/main/java/org/apache/ranger/audit/provider/MiscUtil.java 8ad4616 agents-audit/src/main/java/org/apache/ranger/audit/provider/hdfs/HdfsLogDestination.java 0951c64 agents-audit/src/main/java/org/apache/ranger/audit/queue/AuditBatchQueue.java 5b56c96 agents-audit/src/main/java/org/apache/ranger/audit/queue/AuditFileSpool.java 17ddab9 hbase-agent/src/main/java/org/apache/ranger/services/hbase/client/HBaseClient.java 65e7be6 hdfs-agent/src/main/java/org/apache/ranger/services/hdfs/client/HdfsClient.java bc98d24 hive-agent/src/main/java/org/apache/ranger/authorization/hive/authorizer/RangerHiveAuthorizer.java bfe1891 plugin-kafka/src/main/java/org/apache/ranger/authorization/kafka/authorizer/RangerKafkaAuthorizer.java 1afedd5 plugin-solr/src/main/java/org/apache/ranger/authorization/solr/authorizer/RangerSolrAuthorizer.java 9371eaa ranger-plugin-classloader/src/main/java/org/apache/ranger/plugin/classloader/RangerPluginClassLoaderUtil.java ea18883 Diff: https://reviews.apache.org/r/50591/diff/ Testing ------- Thanks, Ramesh Mani
