rameeshm commented on code in PR #886:
URL: https://github.com/apache/ranger/pull/886#discussion_r3054864907
##########
audit-server/audit-dispatcher/dispatcher-hdfs/src/main/java/org/apache/ranger/audit/server/HdfsDispatcherConfig.java:
##########
@@ -23,39 +23,40 @@
import org.slf4j.LoggerFactory;
/**
- * Loads HDFS consumer-specific configuration files including Hadoop configs.
+ * Loads HDFS dispatcher-specific configuration files including Hadoop configs.
*/
-public class HdfsConsumerConfig extends AuditConfig {
- private static final Logger LOG =
LoggerFactory.getLogger(HdfsConsumerConfig.class);
- private static final String CONFIG_FILE_PATH =
"conf/ranger-audit-consumer-hdfs-site.xml";
- private static final String CORE_SITE_FILE_PATH =
"conf/core-site.xml";
- private static final String HDFS_SITE_FILE_PATH =
"conf/hdfs-site.xml";
- private static volatile HdfsConsumerConfig sInstance;
-
- private HdfsConsumerConfig() {
+public class HdfsDispatcherConfig extends AuditConfig {
Review Comment:
@mneethiraj That is correct, we don't need a separate config for each of the
dispatcher. Addressed this in the review commits set #4
--
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: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
