Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

elasticsearch: set _type=doc #3757

Merged
merged 2 commits into from
Mar 17, 2017
Merged

elasticsearch: set _type=doc #3757

merged 2 commits into from
Mar 17, 2017

Conversation

7AC
Copy link
Contributor

@7AC 7AC commented Mar 15, 2017

Use special _type=doc instead of custom ones. The _type field is deprecated per elastic/elasticsearch#15613.

@7AC 7AC added the review label Mar 15, 2017
@ruflin
Copy link
Member

ruflin commented Mar 16, 2017

Can you link to the elasticsearch / pr of the change to provide some more feedback on why this change is needed?

This must be added to the CHANGELOG.asciidoc as this is a breaking change in case someone was using _type before. This becomes especially obvious for people using filebeat where document_type was mapped to _type.

The `_type` field is deprecated per
elastic/elasticsearch#15613
@ruflin
Copy link
Member

ruflin commented Mar 17, 2017

Can you rebase and squash and update the commit and PR message? This makes it easier for people that stumble over the PR later to find the right info.

@7AC
Copy link
Contributor Author

7AC commented Mar 17, 2017

I updated the first comment to the PR, I have a single commit I don't think I need squashing. Thanks!

@7AC 7AC merged commit bec7603 into elastic:master Mar 17, 2017
@7AC 7AC deleted the fix/elasticsearch_type_doc branch March 17, 2017 21:54
@ruflin
Copy link
Member

ruflin commented Mar 20, 2017

@7AC I see 2 commits in this PR? One is a merge commit.

@tsg tsg added the needs_backport PR is waiting to be backported to other branches. label Apr 24, 2017
@tsg tsg added the v5.5.0 label May 3, 2017
tsg pushed a commit to tsg/beats that referenced this pull request May 3, 2017
The `_type` field is deprecated per
elastic/elasticsearch#15613
(cherry picked from commit bec7603)
@tsg tsg removed the needs_backport PR is waiting to be backported to other branches. label May 3, 2017
monicasarbu pushed a commit that referenced this pull request May 3, 2017
The `_type` field is deprecated per
elastic/elasticsearch#15613
(cherry picked from commit bec7603)
@tsg tsg mentioned this pull request Jul 24, 2017
28 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants