Open dgieselaar opened 1 year ago
Pinging @elastic/apm-ui (Team:APM)
Following yesterday discussion on how to show to the user something has changed in the Based on sample transactions
tooltip
we could use a popover that component would allow us to place a link to documentation, the only thing is the visual styling is different from the tooltip we display now.
There currently is an in-flight PR to set
_doc_count
for transactions and spans, so results on sampled data are automatically upscaled.The expectation is that this should need no work from the UI, but we should verify. In summary, the changes are:
_doc_count
and opaquely upscale resultstrack_total_hits
will still use the real document countWe should check the following:
Ideally we should also have some synthtrace tests. We currently have
sampled()
, but I suggest we replace that withcount(n)
, which would set both{transaction,span}.representative_count
and_doc_count
ton
.