Skip to content

Conversation

@jgowdyelastic
Copy link
Member

@jgowdyelastic jgowdyelastic commented Nov 11, 2025

Fixes #242481

If a data view contains a runtime field, it will be added twice. This has become a problem after a recent Data Grid change.

@jgowdyelastic jgowdyelastic self-assigned this Nov 11, 2025
@jgowdyelastic jgowdyelastic added release_note:fix :ml v9.3.0 v9.2.1 Feature:Data Frame Analytics ML data frame analytics features backport:version Backport to applied version labels labels Nov 11, 2025
@jgowdyelastic jgowdyelastic marked this pull request as ready for review November 11, 2025 15:20
@jgowdyelastic jgowdyelastic requested a review from a team as a code owner November 11, 2025 15:20
@elasticmachine
Copy link
Contributor

Pinging @elastic/ml-ui (:ml)

@peteharverson peteharverson changed the title [ML] Fixing DFA wizard for data view with runtime fields [ML] Data frame analytics: Fixes wizard for data view with runtime fields Nov 11, 2025
Copy link
Contributor

@peteharverson peteharverson left a comment

Choose a reason for hiding this comment

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

Tested with data views with runtime fields and LGTM

Copy link
Contributor

@alvarezmelissa87 alvarezmelissa87 left a comment

Choose a reason for hiding this comment

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

LGTM ⚡

@elasticmachine
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
ml 5.4MB 5.4MB +67.0B

History

cc @jgowdyelastic

@jgowdyelastic jgowdyelastic merged commit 3f68ad6 into elastic:main Nov 12, 2025
15 checks passed
@kibanamachine
Copy link
Contributor

Starting backport for target branches: 9.2

https://github.com/elastic/kibana/actions/runs/19291316842

kibanamachine pushed a commit to kibanamachine/kibana that referenced this pull request Nov 12, 2025
…elds (elastic#242557)

Fixes elastic#242481

If a data view contains a runtime field, it will be added twice. This
has become a problem after a recent Data Grid
[change](elastic/eui#9030).

(cherry picked from commit 3f68ad6)
@kibanamachine
Copy link
Contributor

💚 All backports created successfully

Status Branch Result
9.2

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 Nov 12, 2025
…ime fields (#242557) (#242634)

# Backport

This will backport the following commits from `main` to `9.2`:
- [[ML] Data frame analytics: Fixes wizard for data view with runtime
fields (#242557)](#242557)

<!--- Backport version: 9.6.6 -->

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

<!--BACKPORT [{"author":{"name":"James
Gowdy","email":"[email protected]"},"sourceCommit":{"committedDate":"2025-11-12T08:37:49Z","message":"[ML]
Data frame analytics: Fixes wizard for data view with runtime fields
(#242557)\n\nFixes https://github.com/elastic/kibana/issues/242481\n\nIf
a data view contains a runtime field, it will be added twice. This\nhas
become a problem after a recent Data
Grid\n[change](https://github.com/elastic/eui/pull/9030).","sha":"3f68ad6d92d30a120932eb4ae466c3f9de442f9e","branchLabelMapping":{"^v9.3.0$":"main","^v(\\d+).(\\d+).\\d+$":"$1.$2"}},"sourcePullRequest":{"labels":["release_note:fix",":ml","Feature:Data
Frame Analytics","backport:version","v9.3.0","v9.2.2"],"title":"[ML]
Data frame analytics: Fixes wizard for data view with runtime
fields","number":242557,"url":"https://github.com/elastic/kibana/pull/242557","mergeCommit":{"message":"[ML]
Data frame analytics: Fixes wizard for data view with runtime fields
(#242557)\n\nFixes https://github.com/elastic/kibana/issues/242481\n\nIf
a data view contains a runtime field, it will be added twice. This\nhas
become a problem after a recent Data
Grid\n[change](https://github.com/elastic/eui/pull/9030).","sha":"3f68ad6d92d30a120932eb4ae466c3f9de442f9e"}},"sourceBranch":"main","suggestedTargetBranches":["9.2"],"targetPullRequestStates":[{"branch":"main","label":"v9.3.0","branchLabelMappingKey":"^v9.3.0$","isSourceBranch":true,"state":"MERGED","url":"https://github.com/elastic/kibana/pull/242557","number":242557,"mergeCommit":{"message":"[ML]
Data frame analytics: Fixes wizard for data view with runtime fields
(#242557)\n\nFixes https://github.com/elastic/kibana/issues/242481\n\nIf
a data view contains a runtime field, it will be added twice. This\nhas
become a problem after a recent Data
Grid\n[change](https://github.com/elastic/eui/pull/9030).","sha":"3f68ad6d92d30a120932eb4ae466c3f9de442f9e"}},{"branch":"9.2","label":"v9.2.2","branchLabelMappingKey":"^v(\\d+).(\\d+).\\d+$","isSourceBranch":false,"state":"NOT_CREATED"}]}]
BACKPORT-->

Co-authored-by: James Gowdy <[email protected]>
@mistic mistic removed the v9.2.1 label Nov 12, 2025
eokoneyo pushed a commit to eokoneyo/kibana that referenced this pull request Dec 2, 2025
…elds (elastic#242557)

Fixes elastic#242481

If a data view contains a runtime field, it will be added twice. This
has become a problem after a recent Data Grid
[change](elastic/eui#9030).
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backport:version Backport to applied version labels Feature:Data Frame Analytics ML data frame analytics features :ml release_note:fix v9.2.2 v9.3.0

Projects

None yet

6 participants