Skip to content

[9.1] [Security Solution][Sourcerer] Replace unnecessary useDataViewSpec calls (#224831)#226183

Merged
kibanamachine merged 6 commits intoelastic:9.1from
kibanamachine:backport/9.1/pr-224831
Jul 15, 2025
Merged

[9.1] [Security Solution][Sourcerer] Replace unnecessary useDataViewSpec calls (#224831)#226183
kibanamachine merged 6 commits intoelastic:9.1from
kibanamachine:backport/9.1/pr-224831

Conversation

@kibanamachine
Copy link
Contributor

Backport

This will backport the following commits from main to 9.1:

Questions ?

Please refer to the Backport tool documentation

…lls (elastic#224831)

## Summary

As useDataViewSpec uses are expensive (due to toSpec method in the data
view implementation), this PR
adds in an optional switch to the hook and actually replaces it where we
only need the data view id and not the full spec.

### Testing

Set the FF,

```
xpack.securitySolution.enableExperimental: ['newDataViewPickerEnabled']
```

verify that the app still works correctly (timelines)

### Checklist

Check the PR satisfies following conditions.

Reviewers should verify this PR satisfies this list as well.

- [x] [Unit or functional
tests](https://www.elastic.co/guide/en/kibana/master/development-tests.html)
were updated or added to match the most common scenarios

(cherry picked from commit 711ab70)
@kibanamachine kibanamachine added the backport This PR is a backport of another PR label Jul 2, 2025
@kibanamachine kibanamachine enabled auto-merge (squash) July 2, 2025 12:29
@elasticmachine
Copy link
Contributor

💔 Build Failed

Failed CI Steps

History

cc @lgestc

@kibanamachine kibanamachine merged commit 9a60130 into elastic:9.1 Jul 15, 2025
11 of 12 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backport This PR is a backport of another PR

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants