[ML] Remove beta badge for Field statistics table in Discover

[qn895]

Summary

This PR removes the beta badge for the Field statistics table.

Points of consideration for keeping the beta badge:

• Easier for us to keep collecting more user feedback.
• Potentially switching to using the new random sampler for aggregation for the field statistics table in the next release. Currently, we are pausing this work to match up with the popover ([Discover] New field stats in Discover sidebar popover #139072 and [ML] Match Data Visualizer/Field stats table content with the popover #140667) and to fine-tune the user experience/performance.

Points of consideration for removing the beta badge:

• The field stats table has been available to users since 8.1, and has been in use within ML since 7.x. We should be defining clear criterias for when it can be moved to GA.

Checklist

Delete any items that are not applicable to this PR.

• Any text added follows EUI's writing guidelines, uses sentence case text and includes i18n support
• Documentation was added for features that require explanation or tutorials
• Unit or functional tests were updated or added to match the most common scenarios
• Any UI touched in this PR is usable by keyboard only (learn more about keyboard accessibility)
• Any UI touched in this PR does not create any new axe failures (run axe in browser: FF, Chrome)
• If a plugin configuration key changed, check if it needs to be allowlisted in the cloud and added to the docker list
• This renders correctly on smaller devices using a responsive layout. (You can test this in your browser)
• This was checked for cross-browser compatibility

Risk Matrix

Delete this section if it is not applicable to this PR.

Before closing this PR, invite QA, stakeholders, and other developers to identify risks that should be tested prior to the change/feature release.

When forming the risk matrix, consider some of the following examples and how they may potentially impact the change:

Risk	Probability	Severity	Mitigation/Notes
Multiple Spaces—unexpected behavior in non-default Kibana Space.	Low	High	Integration tests will verify that all features are still supported in non-default Kibana Space and when user switches between spaces.
Multiple nodes—Elasticsearch polling might have race conditions when multiple Kibana nodes are polling for the same tasks.	High	Low	Tasks are idempotent, so executing them multiple times will not result in logical error, but will degrade performance. To test for this case we add plenty of unit tests around this logic and document manual testing procedure.
Code should gracefully handle cases when feature X or plugin Y are disabled.	Medium	High	Unit tests will verify that any feature flag or plugin combination still results in our service operational.
See more potential risk examples

For maintainers

• This was checked for breaking API changes and was labeled appropriately

[elasticmachine]

Pinging @elastic/ml-ui (:ml)

[davismcphee]

Code changes LGTM, gonna add @ninoslavmiskovic for product review

[peteharverson]

My vote is to leave the beta badge on the field statistics tab for the reasons @qn895 outlined above, specifically until we decide how to integrate the random sampler aggregation into the field statistics table - see #138953.

[kertal]

My vote is to leave the beta badge on the field statistics tab for the reasons @qn895 outlined above, specifically until we decide how to integrate the random sampler aggregation into the field statistics table - see #138953.

This makes sense, yes 👍

[kertal]

So #138953 was replaced with #144646, which was merged. added this for visibility

[kertal]

@peteharverson @qn895 so from your team's perspective we could remove the beta badge? thx!

[peteharverson]

@kertal, yes from the ML point of view, we think the beta badge could be removed for 8.7. The field statistics table in Discover doesn't yet use the random sampler (correct me if I'm wrong @qn895) - #144646 only applied it to the Data Visualizer inside the ML plugin. But I don't think this should stop us removing the beta badge from the field statistics table in Discover.

Are you happy for us to remove it for 8.7 @kertal @ninoslavmiskovic ?

[ninoslavmiskovic]

I am okay with it 👍

[kertal]

👍 let's do it

[kibana-ci]

💛 Build succeeded, but was flaky

• Buildkite Build
• Commit: 39fb3d3

Failed CI Steps

• Jest Tests #1
• Jest Tests #7
• Jest Tests #9

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
discover	418.4KB	418.1KB	-242.0B

Unknown metric groups

ESLint disabled in files

id	before	after	diff
osquery	1	2	+1

ESLint disabled line counts

id	before	after	diff
enterpriseSearch	19	21	+2
fleet	61	67	+6
osquery	109	115	+6
securitySolution	445	451	+6
total			+20

Total ESLint disabled count

id	before	after	diff
enterpriseSearch	20	22	+2
fleet	70	76	+6
osquery	110	117	+7
securitySolution	521	527	+6
total			+21

History

• 💔 Build #73556 failed dfc735b

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

cc @qn895

[peteharverson]

Tested and LGTM