Skip to content

[Automatic Import] Refactor useEffect to reduce complexity#194150

Merged
bhapas merged 4 commits intoelastic:mainfrom
bhapas:automatic_import_improvement
Sep 26, 2024
Merged

[Automatic Import] Refactor useEffect to reduce complexity#194150
bhapas merged 4 commits intoelastic:mainfrom
bhapas:automatic_import_improvement

Conversation

@bhapas
Copy link
Contributor

@bhapas bhapas commented Sep 26, 2024

Summary

Closes - #193947

  • Extracted the useGeneration hook into own file use_generation.tsx
  • Created new interfaces for GenerationResults , RunGenerationProps

@bhapas bhapas requested a review from a team as a code owner September 26, 2024 13:39
@bhapas bhapas changed the title [Automatic ImportRefactor useEffect to reduce complexity [Automatic Import] Refactor useEffect to reduce complexity Sep 26, 2024
@bhapas bhapas requested a review from semd September 26, 2024 13:39
@bhapas bhapas marked this pull request as draft September 26, 2024 13:39
@bhapas bhapas self-assigned this Sep 26, 2024
@bhapas bhapas added enhancement New value added to drive a business result backport:prev-minor Team:Security-Scalability Security Integrations Scalability Team Feature:AutomaticImport labels Sep 26, 2024
@bhapas bhapas marked this pull request as ready for review September 26, 2024 13:48
@elasticmachine
Copy link
Contributor

Pinging @elastic/security-scalability (Team:Security-Scalability)

@bhapas bhapas added release_note:skip Skip the PR/issue when compiling release notes and removed enhancement New value added to drive a business result labels Sep 26, 2024
@bhapas bhapas requested a review from semd September 26, 2024 14:02
@bhapas bhapas requested a review from semd September 26, 2024 14:52
Copy link
Contributor

@kgeller kgeller left a comment

Choose a reason for hiding this comment

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

LGTM

Copy link
Contributor

@semd semd left a comment

Choose a reason for hiding this comment

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

LGTM! Thanks for doing this 💯

@bhapas bhapas enabled auto-merge (squash) September 26, 2024 15:56
@kibana-ci
Copy link

💛 Build succeeded, but was flaky

Failed CI Steps

Test Failures

  • [job] [logs] FTR Configs #72 / discover/group3 discover lens vis should show no histogram for non-time-based data views and recover for time-based data views
  • [job] [logs] Jest Integration Tests #2 / when rolling back to an older version kibana should detect that a later version alias exists, and abort

Metrics [docs]

Module Count

Fewer modules leads to a faster build time

id before after diff
integrationAssistant 559 560 +1

Async chunks

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

id before after diff
integrationAssistant 948.8KB 948.8KB +52.0B

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

cc @bhapas

@bhapas bhapas merged commit bcdb0d8 into elastic:main Sep 26, 2024
kibanamachine pushed a commit to kibanamachine/kibana that referenced this pull request Sep 26, 2024
@kibanamachine
Copy link
Contributor

💚 All backports created successfully

Status Branch Result
8.x

Note: Successful backport PRs will be merged automatically after passing CI.

Questions ?

Please refer to the Backport tool documentation

kibanamachine added a commit that referenced this pull request Sep 26, 2024
…4150) (#194194)

# Backport

This will backport the following commits from `main` to `8.x`:
- [[Automatic Import] Refactor useEffect to reduce complexity
(#194150)](#194150)

<!--- Backport version: 9.4.3 -->

### Questions ?
Please refer to the [Backport tool
documentation](https://github.com/sqren/backport)

<!--BACKPORT [{"author":{"name":"Bharat
Pasupula","email":"123897612+bhapas@users.noreply.github.com"},"sourceCommit":{"committedDate":"2024-09-26T16:50:28Z","message":"[Automatic
Import] Refactor useEffect to reduce complexity
(#194150)","sha":"bcdb0d8ede90a99317bddd99cc122dc98107d1a7","branchLabelMapping":{"^v9.0.0$":"main","^v8.16.0$":"8.x","^v(\\d+).(\\d+).\\d+$":"$1.$2"}},"sourcePullRequest":{"labels":["release_note:skip","v9.0.0","backport:prev-minor","Team:Security-Scalability","Feature:AutomaticImport"],"title":"[Automatic
Import] Refactor useEffect to reduce
complexity","number":194150,"url":"https://github.com/elastic/kibana/pull/194150","mergeCommit":{"message":"[Automatic
Import] Refactor useEffect to reduce complexity
(#194150)","sha":"bcdb0d8ede90a99317bddd99cc122dc98107d1a7"}},"sourceBranch":"main","suggestedTargetBranches":[],"targetPullRequestStates":[{"branch":"main","label":"v9.0.0","branchLabelMappingKey":"^v9.0.0$","isSourceBranch":true,"state":"MERGED","url":"https://github.com/elastic/kibana/pull/194150","number":194150,"mergeCommit":{"message":"[Automatic
Import] Refactor useEffect to reduce complexity
(#194150)","sha":"bcdb0d8ede90a99317bddd99cc122dc98107d1a7"}}]}]
BACKPORT-->

Co-authored-by: Bharat Pasupula <123897612+bhapas@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Feature:AutomaticImport release_note:skip Skip the PR/issue when compiling release notes Team:Security-Scalability Security Integrations Scalability Team v8.16.0 v9.0.0

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants