Github user cestella commented on the issue:
https://github.com/apache/metron/pull/636
This is good work! Thanks, Ryan. We desperately need this abstraction. I
think the confusion may be that this PR is the general abstraction as well as a
concrete implementation for one index. I think, as we found with the indexing
topology, that solr and ES won't be able to exist within the same project due
to classpath issues (note we have 2 separate projects for solr and ES to
isolate the dependencies completely). I would think that the ES specific stuff
may should be separated into a separate project (`metron-rest-elasticsearch`)
where the abstraction should exist in `metron-rest`.
---
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.
---
