Github user merrimanr commented on the issue:

    https://github.com/apache/metron/pull/827
  
    @JonZeolla I think we should just remove it for now and I've done that in 
the latest commit.  I'm not entirely clear on what we should expect in this 
case and we don't need it right now anyways.  Better to implement it the right 
way once we are clear on how it should behave in my opinion.
    
    Good catch @justinleet!  I did not think of this test case and to my 
surprise the Elasticsearch API returns ALL field mappings if you pass in an 
empty array of indices.  I will fix this and add a test case.  You are correct 
in that we should expect correctly formatted ip addresses in "facetCounts".  
Before this PR they would have been in long format when you include an index 
that does not have a mapping for the ip_src_addr field.


---

Reply via email to