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

Increase index pattern select limit to 1000 #92093

Merged

Conversation

kindsun
Copy link
Contributor

@kindsun kindsun commented Feb 19, 2021

Bumps the index pattern select component to a default of 1000 and allows props to be passed in to override the default limit. Resolves #86915.

There are at least 3 potential solutions to this issue:

  1. Use Search after to get paginated results. This seems like overkill for index patterns, and to this point, the index patterns service which this component depends on isn't doing this.
  2. [Implemented solution] Bump the request amount to a higher number than 100. Currently the index patterns service maintains a saved objects cache of 10,000 so I don't think it's unreasonable to submit find requests of 1,000.
  3. Potentially in addition to number 2, we could maintain a find cache in the index patterns service similar to how we do with the standard saved objects cache. This cache would only get used when the same search term is submitted more than one time in a row. Usefulness seems negligible but I'm open to adding this.

@kindsun kindsun requested a review from a team as a code owner February 19, 2021 21:50
@kindsun kindsun added [Deprecated-Use Team:Presentation]Team:Geo Former Team Label for Geo Team. Now use Team:Presentation v7.12.0 v7.13.0 v7.8.0 labels Feb 19, 2021
@elasticmachine
Copy link
Contributor

Pinging @elastic/kibana-gis (Team:Geo)

@kindsun kindsun added the release_note:skip Skip the PR/issue when compiling release notes label Feb 19, 2021
@nreese nreese added v8.0.0 and removed v7.8.0 labels Feb 20, 2021
Copy link
Contributor

@mattkime mattkime left a comment

Choose a reason for hiding this comment

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

code changes lgtm. A long term solution would be a way to express that we want to load {x} number of {criteria} index patterns. But that seems like overkill for the immediate need.

Copy link
Contributor

@thomasneirynck thomasneirynck left a comment

Choose a reason for hiding this comment

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

code review lgtm

@kindsun
Copy link
Contributor Author

kindsun commented Feb 24, 2021

@elasticmachine merge upstream

@kindsun kindsun force-pushed the bump-up-index-pattern-select-component-limit branch from 4d73553 to 110a705 Compare February 24, 2021 18:06
@kindsun
Copy link
Contributor Author

kindsun commented Feb 24, 2021

@elasticmachine merge upstream

@kibanamachine
Copy link
Contributor

💚 Build Succeeded

Metrics [docs]

Async chunks

Total size of all lazy-loaded chunks that will be downloaded as the user navigates the app

id before after diff
data 217.4KB 217.5KB +81.0B

History

To update your PR or re-run it, just comment with:
@elasticmachine merge upstream

@kindsun kindsun merged commit 91d03f0 into elastic:master Feb 24, 2021
@kindsun kindsun deleted the bump-up-index-pattern-select-component-limit branch February 24, 2021 23:01
kindsun pushed a commit to kindsun/kibana that referenced this pull request Feb 24, 2021
kindsun pushed a commit to kindsun/kibana that referenced this pull request Feb 24, 2021
kindsun pushed a commit that referenced this pull request Feb 25, 2021
Co-authored-by: Kibana Machine <[email protected]>

Co-authored-by: Kibana Machine <[email protected]>
gmmorris added a commit to gmmorris/kibana that referenced this pull request Feb 25, 2021
* master: (38 commits)
  Fixes Cypress flake by adding pipe, click, and should (elastic#92762)
  [Discover] Fix filtering selected sidebar fields (elastic#91828)
  [ML] Fixes positions of calendar arrow buttons in start datafeed modal (elastic#92625)
  [dev/build_ts_refs] check that commit in outDirs matches mergeBase (elastic#92513)
  add dep on `@kbn/config` so it is built first
  [Expressions] [Lens] Add id and copyMetaFrom arg to mapColumn fn + add configurable onError argument to math fn (elastic#90481)
  [Lens] Fix Workspace hidden when using Safari (elastic#92616)
  [Lens] Fixes vertical alignment validation messages (elastic#91878)
  forbid x-elastic-product-origin header in elasticsearch configuration (elastic#92359)
  [Security Solution][Detections] Set default indicator path to reduce friction with new filebeat modules (elastic#92081)
  [ILM][Accessibility] Added A11y test for ILM new policy form. (elastic#92570)
  [Security Solution][Exceptions] - Fixes exceptions builder UI where invalid values can cause overwrites of other values (elastic#90634)
  Automatically generated Api documentation (elastic#86232)
  Increase index pattern select limit to 1000 (elastic#92093)
  [core.logging] Add RewriteAppender for filtering LogMeta. (elastic#91492)
  [Security Solution][Detection Rules] Update prebuilt rule threats to match schema (elastic#92281)
  [Security Solutions][Detection Engine] Fixes bug with not being able to duplicate indicator matches (elastic#92565)
  [Dashboard] Export appropriate references from byValue panels (elastic#91567)
  [Upgrade Assistant] Align code between branches (elastic#91862)
  [Security Solution][Case] Fix alerts push (elastic#91638)
  ...
@thomasneirynck thomasneirynck added release_note:fix and removed release_note:skip Skip the PR/issue when compiling release notes labels Feb 25, 2021
kindsun pushed a commit that referenced this pull request Feb 26, 2021
Co-authored-by: Kibana Machine <[email protected]>

Co-authored-by: Kibana Machine <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Maps] Won't show all available index patterns if there are more than 100 index patterns
6 participants