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

Remove mapper attachments plugin #20416

Merged
merged 1 commit into from Sep 19, 2016

Conversation

Projects
None yet
3 participants
@dadoonet
Member

dadoonet commented Sep 12, 2016

We now have in 5.0.0 ingest-attachment plugin.
We can remove mapper-attachments plugin for 6.0.

Closes #18837.

@dakrone

This comment has been minimized.

Show comment
Hide comment
@dakrone

dakrone Sep 15, 2016

Member

LGTM!

Member

dakrone commented Sep 15, 2016

LGTM!

Remove mapper attachments plugin
We now have in 5.0.0 `ingest-attachment` plugin.
We can remove `mapper-attachments` plugin for 6.0.

Closes #18837.

@dadoonet dadoonet merged commit dfd1eeb into elastic:master Sep 19, 2016

1 of 2 checks passed

elasticsearch-ci Build finished.
Details
CLA Commit author is a member of Elasticsearch
Details

@dadoonet dadoonet deleted the dadoonet:pr/remove-mapper-attachments branch Sep 19, 2016

ruflin added a commit to ruflin/Elastica that referenced this pull request Sep 14, 2017

Removed mapper-attachments plugin. Now use the ingest-attachment plug…
…in (#1375)

[Mapper Attachment plugin has been removed](elastic/elasticsearch#20416) Use Ingest-attachment plugin and attachment processors with pipeline to ingest new documents.

the flow for ingesting a file into Elasticsearch changed a bit :

- u should create a Pipeline with an *Attachment Processor*
- add a file to the document
- ad the document to the index using a query string param of this format : **pipeline=name_of_the_pipeline**
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment