Github user cestella commented on the issue: https://github.com/apache/metron/pull/831 @nickwallen Whoops, sorry, I totally misunderstood. This PR *is* how *I* would like it. I'm just going to copy this here to justify my position: ``` The reason why I preferred to do that was that when we do a better job of including Solr or other indices, prefixing the variables elasticsearch will be confusing. The flux files are generic to the writer type for the random access indices. I can understand the confusion for the user given we only really support HDFS and elasticsearch right now, which is why I changed the display names and kept the command line clients named as they were before. Eventually, when we provide actual proper extension points to the indices, we should genericize the display names and scripts IMO. ``` I hope that clarifies my position on the matter (or at least why I made it the way it is). If there are other thoughts, I'd like to be able to discuss them. @mmiklavc chimed in in support, but I'd like to hear differing opinions if my argument is not persuasive.
---