[DataViews] Make use of DataViewLazy in DataViews API #176426
Assignees
Labels
Feature:Data Views
Data Views code and UI - index patterns before 8.0
impact:high
Addressing this issue will have a high level of impact on the quality/strength of our product.
loe:small
Small Level of Effort
Team:DataDiscovery
Discover, search (e.g. data plugin and KQL), data views, saved searches. For ES|QL, use Team:ES|QL.
Comments
kertal
added
the
Team:DataDiscovery
|
Pinging @elastic/kibana-data-discovery (Team:DataDiscovery) |
kertal
added
the
Feature:Data Views
kertal
added
impact:high
mattkime
added a commit
that referenced
this issue
Apr 3, 2024
## Summary Part 1 of #178594 Initial implementation of DataViewLazy class - an alternative DataView implementation that doesn't load the full field list on initial creation. ``` Implements DataViewLazy (variations from DataView class) - getFields ({ mapped = true, scripted = true, runtime = true, fieldName, forceRefresh = false, unmapped, indexFilter, metaFields = true }) => Promise<{ getFieldMap: () => Record<string, DataViewField>; getFieldMapSorted: () => Record<string, DataViewField> }> - getRuntimeFields({ fieldName = ['*'] }) => Record<string, DataViewField> - getScriptedFields({ fieldName = ['*'] }) => Record<string, DataViewField> - getFieldByName (string) => Promise<DataViewField> - toSpec (same params as getFields) => Promise<DataViewSpec> other notable changes using DataViewLazy over DataView - won't throw if index pattern fails to match - by default toSpec doesn't return fields, must pass in parameters for requested fields --- DataViews service additions - toDataView: (toDataView: DataViewLazy) => Promise<DataView>; - toDataViewLazy: (dataView: DataView) => Promise<DataViewLazy>; - getAllDataViewLazy: () => Promise<DataViewLazy[]>; - getDataViewLazy: (id: string) => Promise<DataViewLazy>; - createDataViewLazy: (spec: DataViewSpec) => Promise<DataViewLazy>; - createAndSaveDataViewLazy: (spec: DataViewSpec, override?: boolean) => Promise<DataViewLazy>; - getDefaultDataViewLazy: () => Promise<DataViewLazy | null>; ``` Closes #176426 and #167750 --------- Co-authored-by: kibanamachine <42973632+kibanamachine@users.noreply.github.com>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Use DataViewLazy in DataViews REST APIs
Very simple initial use case for DataViewLazy. There won't be any performance improvements since the APIs almost entirely report the entire field list.
The text was updated successfully, but these errors were encountered: