Remove references to document_type in the docs #8603
Conversation
|
Pushed additional changes for issues identified in #8606. Updated the examples because some changes have happened to the event structure. Also added an updated screen capture for the tutorial because it was easy for me to grab the screen while I was updating the example. |
dedemorton
force-pushed
the
remove_document_type
branch
from
3ecebfc to
ac419f8
Compare
| "host" => "My-MacBook-Pro.local", | ||
| "prospector" => { | ||
| "type" => "log" |
There was a problem hiding this comment.
@dedemorton my point here was, didn't Beats, in general, start defaulting type => doc instead of log in 6.0? elastic/beats#3757
There was a problem hiding this comment.
Unless the prospector "type" is different from the default "doc type". You may be understand it better than I do.
There was a problem hiding this comment.
Yes, that's the difference. If you look at the full output, you'll see that the _type field is set to doc as expected. The type filed on line 174 is the prospector type.
|
@dedemorton LGTM, thank you. |
|
We good to merge this one? |
Doc changes related to logstash-plugins/logstash-output-elasticsearch#691
Removes
document_typefrom LS configuration examples in the static docs. Thedocument_typeis now set automatically todoc, which is consistent with what the Beats dashboards expect.I've searched the static docs for
document_typeand_typeand have found no other instances, so I'm assuming there is no additional impact to the static docs unless I'm overlooking something.I'll open separate PRs to address any references to
document_typein the plugin docs.