[ML] Explain Log Rate Spikes: merge duplicate spikeAnalysisTable components

[alvarezmelissa87]

Summary

This PR removes the SpikeAnalysisTableExpandedRow component and uses the SpikeAnalysisTable component as they were almost exactly the same. The small difference has been moved to the SpikeAnalysisTable component.

Checklist

Delete any items that are not applicable to this PR.

• Unit or functional tests were updated or added to match the most common scenarios

[elasticmachine]

Pinging @elastic/ml-ui (:ml)

[walterra]

LGTM!

[peteharverson]

Adding for reference, as this happens on main too, but here expand / collapse works on the wrong row:

[qn895]

LGTM

[peteharverson]

LGTM

[walterra]

@peteharverson It's a known issue, we can only fix it once this PR is merged, it introduces unique ids for the groups which is required to make the row expansions work properly.

[kibana-ci]

💚 Build Succeeded

• Buildkite Build
• Commit: 0aac617

Metrics [docs]

Module Count

Fewer modules leads to a faster build time

id	before	after	diff
aiops	499	498	-1

Async chunks

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

id	before	after	diff
aiops	760.5KB	755.4KB	-5.1KB

History

• 💚 Build #74018 succeeded 94a0ded

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

cc @alvarezmelissa87