Skip to content

Conversation

@ppf2
Copy link
Contributor

@ppf2 ppf2 commented Oct 12, 2017

Some users are not expecting create index api settings/mappings to take precedence over index template. This is added to clarify the behavior.

Copy link
Member

@jasontedor jasontedor left a comment

Choose a reason for hiding this comment

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

I left one comment.


NOTE: Templates are only applied at index creation time. Changing a template
will have no impact on existing indices.
will have no impact on existing indices. When using the create index api, the settings/mappings
Copy link
Member

Choose a reason for hiding this comment

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

Would you capitalize API please?

Copy link
Member

@jasontedor jasontedor left a comment

Choose a reason for hiding this comment

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

LGTM.

@jasontedor jasontedor merged commit 1125bc6 into master Oct 12, 2017
@ppf2 ppf2 deleted the ppf2-clarify_index_template branch October 12, 2017 21:49
jasontedor pushed a commit that referenced this pull request Oct 12, 2017
This commit clarifies the interaction between settings specified in a
create index request, and those that would come from any templates that
apply to the create index request.

Relates #26994
jasontedor pushed a commit that referenced this pull request Oct 12, 2017
This commit clarifies the interaction between settings specified in a
create index request, and those that would come from any templates that
apply to the create index request.

Relates #26994
jasontedor pushed a commit that referenced this pull request Oct 12, 2017
This commit clarifies the interaction between settings specified in a
create index request, and those that would come from any templates that
apply to the create index request.

Relates #26994
@jasontedor
Copy link
Member

Thanks so much @ppf2. I merged and backported and added labels.

@ppf2
Copy link
Contributor Author

ppf2 commented Oct 12, 2017

Thx for the review+merge!

@lcawl lcawl added v6.0.0-rc2 and removed v6.0.0 labels Oct 30, 2017
@lcawl lcawl removed the v6.1.0 label Dec 12, 2017
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.

5 participants