Github user nickwallen commented on the issue:

    https://github.com/apache/metron/pull/1218
  
    > @mraliagha: So if es.document.id is not provided, as the default, doc id 
won't be send to ES indexing, right?
    
    Yes, exactly.  
    
    > @mraliagha: I guess it would be also nice to provide some guidance on how 
document ID should be defined (in the case of custom ID). Otherwise, users may 
create some serious issues with the indexing and search throughput.
    
    I am just providing the **capability** for advanced users to define their 
own doc ID, primarily based on your feedback in METRON-1677.  (It also provides 
a nice way to support backwards compatibility, which is the main reason that I 
took this approach.) 
    
    If you have any advice to offer, feel free to offer it and we can include 
it in the docs.  Other than that, I am not sure what I can do besides add a 
big, bold warning to the docs that says create your own doc ID at your own risk.



---

Reply via email to