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

Feature/display langs in status report #3552

Merged
merged 6 commits into from
Jul 24, 2023

Conversation

felipeelia
Copy link
Member

This should be reviewed only after #3551 is merged.

Description of the Change

As it could be interesting to show more index settings in the status report, this PR changes EP so instead of caching a single index setting value (index.mapping.total_fields.limit) it caches the whole array (~4kb max.)

Note: As the transient is set using an expiration time, it won't be added an autoload option.

Changelog Entry

Added - Indices' language settings in status reports

Credits

Props @felipeelia

Checklist:

  • I agree to follow this project's Code of Conduct.
  • I have updated the documentation accordingly.
  • I have added tests to cover my change.
  • All new and existing tests pass.

@felipeelia felipeelia added this to the 4.7.0 milestone Jul 21, 2023
@felipeelia felipeelia merged commit 4973ba0 into develop Jul 24, 2023
@felipeelia felipeelia deleted the feature/display-langs-in-status-report branch July 24, 2023 20:18
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.

1 participant