Skip to content

Conversation

wajihaparvez
Copy link
Contributor

Closes #372
Preview here

Summary

What I changed:

  • Combined content from the two mapped pages and refined it
  • Renamed the page to a more user friendly title. Let me know what you think!
  • Deleted the linked raw migrated files

I'll delete code comments after the review

% - [x] ./raw-migrated-files/elasticsearch/elasticsearch-reference/ignore_missing_component_templates.md
% - [x] ./raw-migrated-files/elasticsearch/elasticsearch-reference/_usage_example.md

When an index template references a component template that might not exist, the `ignore_missing_component_templates` configuration option can be used. With this option, every time a data stream is created based on the index template, the existence of the component template will be checked. If it exists, it will used to form the index’s composite settings. If it is missing, it will be ignored.
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
When an index template references a component template that might not exist, the `ignore_missing_component_templates` configuration option can be used. With this option, every time a data stream is created based on the index template, the existence of the component template will be checked. If it exists, it will used to form the index’s composite settings. If it is missing, it will be ignored.
When an index template references a component template that might not exist, the `ignore_missing_component_templates` configuration option can be used. With this option, every time a data stream is created based on the index template, the existence of the component template will be checked. If it exists, it will used to form the index’s composite settings. If it is missing, it will be ignored.

Perfectly good. Just one suggestion: We often aim to address the user directly with active sentences. e.g., "You can use the abc configuration option..." I wouldn't bother changing it in this case, but I noticed it a couple of times in your PRs so just wanted to mention it.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Thank you, @kilfoyle! For this sentence specifically, the old page phrased it this way and I wasn't 100% sure that this configuration option is something a user can directly implement. But I'll remember for next time!

Copy link
Contributor

@kilfoyle kilfoyle left a comment

Choose a reason for hiding this comment

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

LGTM! 🏎️

@wajihaparvez wajihaparvez enabled auto-merge (squash) February 19, 2025 14:41
@wajihaparvez wajihaparvez merged commit c0d949f into elastic:main Feb 19, 2025
3 of 4 checks passed
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