Github user merrimanr commented on a diff in the pull request:
https://github.com/apache/metron/pull/635#discussion_r125913966
--- Diff:
metron-platform/metron-elasticsearch/src/test/java/org/apache/metron/elasticsearch/integration/ElasticsearchIndexingIntegrationTest.java
---
@@ -102,11 +102,16 @@ public void setAdditionalProperties(Properties
topologyProperties) {
topologyProperties.setProperty("es.clustername", "metron");
topologyProperties.setProperty("es.port", "9300");
topologyProperties.setProperty("es.ip", "localhost");
- topologyProperties.setProperty("indexing.writer.class.name",
"org.apache.metron.elasticsearch.writer.ElasticsearchWriter");
+ topologyProperties.setProperty("indexing_writer_class_name",
"org.apache.metron.elasticsearch.writer.ElasticsearchWriter");
}
@Override
public String cleanField(String field) {
return field;
}
--- End diff --
I think it's a valid concern. I'm also not sure what the ideal alternative
is. At this point I feel like Metron has succumbed to being tightly coupled
with Ambari.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at [email protected] or file a JIRA ticket
with INFRA.
---
