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

Merge dynamic known fields, added new cron task #1276

Merged
merged 9 commits into from
Mar 1, 2019
Merged

Conversation

jesusgn90
Copy link
Contributor

Changes:

  • The way we refresh an index pattern has been changed. Now we are fetching all the dynamic fields that we may be missing and it merges those fields with our known fields.
  • Added a new cron task in our server side, it refreshes all valid index patterns every 2 minutes.
  • Removed refreshing action per every parent state change.
  • Keep the refreshing action for all the index patterns if kbn-vis fails due to missing fields*.
  • Keep the refreshing action for all the index patterns if the app health check is fired*.
  • Keep the refreshing action for the selected pattern only when using the index pattern selector*.
  • Wrapped into a double try/catch block, so the server won't be crashed under any error that this feature may throw.

* Now, the refreshing action is using the merging feature described above

@jesusgn90 jesusgn90 added the type/enhancement Enhancement issue label Mar 1, 2019
@jesusgn90 jesusgn90 self-assigned this Mar 1, 2019
@jesusgn90 jesusgn90 merged commit 1a873df into 3.9-6.6 Mar 1, 2019
@jesusgn90 jesusgn90 deleted the auto-known-fields branch March 1, 2019 16:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type/enhancement Enhancement issue
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant