Skip to content

Implement _source parameter on action and doc lines. #4339

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

Merged
merged 9 commits into from
Feb 10, 2020

Conversation

codebrain
Copy link
Contributor

The documentation (https://www.elastic.co/guide/en/elasticsearch/reference/current/docs-bulk.html) indicates that _source is valid on both the action and the doc lines (see the JSON example at the bottom of the page).

{ "update" : {"_id" : "3", "_index" : "index1", "_source" : true} }
{ "doc" : {"field" : "value"} }
{ "update" : {"_id" : "4", "_index" : "index1"} }
{ "doc" : {"field" : "value"}, "_source": true}

This PR is in draft whilst it is ratified if there is any precendence in parameters being set, or if the parameters are preferred on the action line or doc line.

Copy link
Contributor

@russcam russcam left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Left a comment about the _source property type

@russcam
Copy link
Contributor

russcam commented Feb 5, 2020

@codebrain let me know when this is ready to review again

@codebrain codebrain marked this pull request as ready for review February 7, 2020 04:26
@codebrain
Copy link
Contributor Author

@codebrain let me know when this is ready to review again

Ready @russcam!

Copy link
Contributor

@russcam russcam left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I've left some additional comments

Copy link
Contributor

@russcam russcam left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Left some additional comments

Copy link
Contributor

@russcam russcam left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, one small comment about a method that is no longer needed 👍

@codebrain codebrain merged commit e4c1d35 into master Feb 10, 2020
@Mpdreamz Mpdreamz deleted the feature/bulk-api-source branch February 10, 2020 10:34
@russcam
Copy link
Contributor

russcam commented Feb 20, 2020

@codebrain have the Bulk Update supporting _source changes from this PR been ported to 7.x? If so, #4337 can be closed.

@codebrain
Copy link
Contributor Author

It has not been ported to 7.x yet.

russcam pushed a commit that referenced this pull request Feb 24, 2020
Implement _source parameter on action and doc lines.

(cherry picked from commit e4c1d35)
russcam pushed a commit that referenced this pull request Feb 24, 2020
Implement _source parameter on action and doc lines.

(cherry picked from commit e4c1d35)
(cherry picked from commit 3665683)
russcam added a commit that referenced this pull request Feb 24, 2020
Implement _source parameter on action and doc lines.

(cherry picked from commit e4c1d35)
russcam added a commit that referenced this pull request Feb 24, 2020
Implement _source parameter on action and doc lines.

(cherry picked from commit e4c1d35)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants