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

Propagate percolate mapping changes to cluster state #5776

Conversation

martijnvg
Copy link
Member

If the during percolating a new field was introduced in the local mapping service, then those changes should be updated in cluster state of the master as well.

…ping service, then those changes should be updated in cluster state of the master as well.
@kimchy
Copy link
Member

kimchy commented Apr 11, 2014

LGTM, +1 on later brining the duplicate code into the same place.

@s1monw
Copy link
Contributor

s1monw commented Apr 14, 2014

LGTM +1 to push

martijnvg added a commit that referenced this pull request Apr 15, 2014
If the during percolating a new field was introduced
in the local mapping service, then those changes should
be updated in cluster state of the master as well.

Closes #5776
martijnvg added a commit that referenced this pull request Apr 15, 2014
If the during percolating a new field was introduced
in the local mapping service, then those changes should
be updated in cluster state of the master as well.

Closes #5776
@martijnvg martijnvg closed this in 202b1e2 Apr 15, 2014
martijnvg added a commit that referenced this pull request Apr 15, 2014
If the during percolating a new field was introduced
in the local mapping service, then those changes should
be updated in cluster state of the master as well.

Closes #5776
@s1monw
Copy link
Contributor

s1monw commented Apr 15, 2014

@martijnvg I pushed this to all relevant branches

@martijnvg
Copy link
Member Author

Thanks Simon!

On 15 April 2014 19:35, Simon Willnauer notifications@github.com wrote:

@martijnvg https://github.com/martijnvg I pushed this to all relevant
branches


Reply to this email directly or view it on GitHubhttps://github.com//pull/5776#issuecomment-40475896
.

Met vriendelijke groet,

Martijn van Groningen

@martijnvg martijnvg deleted the bug/percolate_propogate_mapping_change branch May 18, 2015 23:32
@clintongormley clintongormley added the :Search/Percolator Reverse search: find queries that match a document label Jun 7, 2015
mute pushed a commit to mute/elasticsearch that referenced this pull request Jul 29, 2015
If the during percolating a new field was introduced
in the local mapping service, then those changes should
be updated in cluster state of the master as well.

Closes elastic#5776
mute pushed a commit to mute/elasticsearch that referenced this pull request Jul 29, 2015
If the during percolating a new field was introduced
in the local mapping service, then those changes should
be updated in cluster state of the master as well.

Closes elastic#5776
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
>bug :Search/Percolator Reverse search: find queries that match a document v1.0.3 v1.1.1 v1.2.0 v2.0.0-beta1
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants