Skip to content

[ML] Removes median line length categorization check#243827

Merged
jgowdyelastic merged 6 commits intoelastic:mainfrom
jgowdyelastic:removing-median-line-length-categorization-check
Nov 25, 2025
Merged

[ML] Removes median line length categorization check#243827
jgowdyelastic merged 6 commits intoelastic:mainfrom
jgowdyelastic:removing-median-line-length-categorization-check

Conversation

@jgowdyelastic
Copy link
Member

@jgowdyelastic jgowdyelastic commented Nov 21, 2025

Removes the median length for field values check in the categorization field validation checks.

These checks are performed in the categorization anomaly detection job wizard, the Log Pattern Analysis AIOps page in ML and the DSL version of pattern analysis in discover

Closes #243822

@jgowdyelastic jgowdyelastic self-assigned this Nov 24, 2025
@jgowdyelastic jgowdyelastic marked this pull request as ready for review November 24, 2025 12:12
@jgowdyelastic jgowdyelastic requested a review from a team as a code owner November 24, 2025 12:12
@elasticmachine
Copy link
Contributor

Pinging @elastic/ml-ui (:ml)

@elasticmachine
Copy link
Contributor

💚 Build Succeeded

Metrics [docs]

Module Count

Fewer modules leads to a faster build time

id before after diff
aiops 608 607 -1
ml 2812 2811 -1
total -2

Async chunks

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

id before after diff
aiops 527.7KB 527.4KB -299.0B
ml 5.6MB 5.6MB -301.0B
total -600.0B
Unknown metric groups

API count

id before after diff
@kbn/ml-category-validator 37 35 -2

History

cc @jgowdyelastic

Copy link
Contributor

@KodeRad KodeRad left a comment

Choose a reason for hiding this comment

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

Code LGTM ⚡

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 and LGTM

@jgowdyelastic jgowdyelastic merged commit 2b7b762 into elastic:main Nov 25, 2025
13 checks passed
seanrathier pushed a commit to seanrathier/kibana that referenced this pull request Nov 25, 2025
Removes the median length for field values check in the categorization
field validation checks.

These checks are performed in the categorization anomaly detection job
wizard, the Log Pattern Analysis AIOps page in ML and the DSL version of
pattern analysis in discover

Closes elastic#243822
eokoneyo pushed a commit to eokoneyo/kibana that referenced this pull request Dec 2, 2025
Removes the median length for field values check in the categorization
field validation checks.

These checks are performed in the categorization anomaly detection job
wizard, the Log Pattern Analysis AIOps page in ML and the DSL version of
pattern analysis in discover

Closes elastic#243822
JordanSh pushed a commit to JordanSh/kibana that referenced this pull request Dec 9, 2025
Removes the median length for field values check in the categorization
field validation checks.

These checks are performed in the categorization anomaly detection job
wizard, the Log Pattern Analysis AIOps page in ML and the DSL version of
pattern analysis in discover

Closes elastic#243822
@peteharverson peteharverson changed the title [ML] Removing median line length categorization check Dec 15, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backport:skip This PR does not require backporting Feature:Anomaly Detection ML anomaly detection :ml release_note:enhancement v9.3.0

4 participants