-
Notifications
You must be signed in to change notification settings - Fork 25.3k
[Docs] Includes information about available index settings in Serverless #129704
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
Conversation
is path in totally the wrong place??? also maybe indexing pressure I guess |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
overall, looks really good.
when i saw the mapping/metadata fields were impacted I got a little nervous - do those references need an overall note on them? wondering how much of the content is deployment-agnostic and how much of it follows this restriction pattern.
approving to unblock
docs/reference/elasticsearch/configuration-reference/data-stream-lifecycle-settings.md
Show resolved
Hide resolved
docs/reference/elasticsearch/configuration-reference/data-stream-lifecycle-settings.md
Show resolved
Hide resolved
docs/reference/elasticsearch/index-settings/_snippets/serverless-availability.md
Show resolved
Hide resolved
Pinging @elastic/es-docs (Team:Docs) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I typed these comments days ago and forgot to submit
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
lgtm
I've updated the serverless introductory paragraph due to that. Very good point, now it's aligned with the upcoming change in elastic/docs-content#1728 |
@eedugon thanks for your great work! this is awesome and I love it! Only one small thing, personal feeling, I found the badge |
@kunisen we are working on the badge designs with UX now. We need to balance badge visibility with making the page too noisy (because we use these same badges everywhere), but you'll see changes to that experience in the coming months as our eng team finds time to address :) |
…ess (elastic#129704) * index settings for serverless information added * comment removed * Apply suggestions from code review Co-authored-by: shainaraskas <[email protected]> * serverless applies removed from coerce doc * eck applies addded to config reference landing * toc updated * index settings landing page aligned with toc * serverless doc intro updated --------- Co-authored-by: shainaraskas <[email protected]>
This Docs PR starts addressing elastic/docs-content#79
The main change is to introduce and show the list of index-level settings available in Serverless. For this purpose:
Serverless
badges in relevant settings, next to the name.Some screenshots for ilustrative purposes:
refresh_interval
setting has been updated as it has a different default in serverless.There are other minor changes / refinement done while reviewing all the content, such as:
dynamic
index settings were pointing to the cluster settings API doc. Fixed.shard allocation
section.general index settings
had some inconsistencies with the conceptindex modules
.