Skip to content

Update refresh.asciidoc for typo and clarification of performed operations #113312

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.

Sign up for GitHub

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

Jump to bottom
Open
wants to merge 1 commit into
base: 8.15
Choose a base branch
from
Open

Update refresh.asciidoc for typo and clarification of performed operations #113312

wants to merge 1 commit into from

Conversation

mrklaney
Copy link
Contributor

Added two words.

  1. "write" operations... - not ALL operations performed on an index are "available for search." (For example, _shrink index resulst are not searchable)
  2. "NOT" received one search... - this typo gives the opposite, incorrect meaning
  • Have you signed the contributor license agreement?
  • Have you followed the contributor guidelines?
  • If submitting code, have you built your formula locally prior to submission with gradle check?
  • If submitting code, is your pull request against main? Unless there is a good reason otherwise, we prefer pull requests against main and will backport as needed.
  • If submitting code, have you checked that your submission is for an OS and architecture that we support?
  • If you are submitting this code for a class then read our policy for that.

@mrklaney
Update refresh.asciidoc
7cbf080 
Added two words.  
1.  "write" operations... - not ALL operations performed on an index are "available for search."  (For example, _shrink index resulst are not searchable)   
2. "NOT" received one search... - this typo gives the opposite, incorrect meaning
@github-actions GitHub Actions
Copy link
Contributor

Documentation preview:

@elasticsearchmachine
Copy link
Collaborator

@mrklaney please enable the option "Allow edits and access to secrets by maintainers" on your PR. For more information, see the documentation.

@elasticsearchmachine elasticsearchmachine added needs:triage Requires assignment of a team area label v8.15.3 external-contributor Pull request authored by a developer outside the Elasticsearch team labels Sep 20, 2024
gbanasiak
gbanasiak reviewed Sep 23, 2024
View reviewed changes
docs/reference/indices/refresh.asciidoc
// tag::refresh-interval-default[]
By default, Elasticsearch periodically refreshes indices every second, but only on
indices that have received one search request or more in the last 30 seconds.
indices that have NOT received one search request or more in the last 30 seconds.
Copy link
Contributor

Choose a reason for hiding this comment

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

I don't think that's accurate. I think there are no refreshes in idle shards.

@gbanasiak gbanasiak added the :Data Management/Indices APIs APIs to create and manage indices and templates label Sep 23, 2024
@elasticsearchmachine elasticsearchmachine added Team:Data Management Meta label for data/management team and removed needs:triage Requires assignment of a team area label labels Sep 23, 2024
@elasticsearchmachine
Copy link
Collaborator

Pinging @elastic/es-data-management (Team:Data Management)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@gbanasiak gbanasiak gbanasiak left review comments

At least 1 approving review is required to merge this pull request.

Assignees

No one assigned

Labels

:Data Management/Indices APIs APIs to create and manage indices and templates external-contributor Pull request authored by a developer outside the Elasticsearch team Team:Data Management Meta label for data/management team v8.15.4

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@mrklaney @elasticsearchmachine @gbanasiak