github-mirrors/kibana
1
0
Fork 0
mirror of https://github.com/elastic/kibana.git synced 2025-04-24 09:48:58 -04:00
Code Issues Projects Releases Packages Wiki Activity

[ML] Fix Index data visualizer doc count when time field is not defined (#142409) (#142532)

Browse source 
Co-authored-by: Kibana Machine <42973632+kibanamachine@users.noreply.github.com>
(cherry picked from commit fdba8d3a56)

Co-authored-by: Quynh Nguyen (Quinn) <43350163+qn895@users.noreply.github.com>
This commit is contained in:
Kibana Machine 2022-10-03 16:25:04 -06:00 • committed by GitHub
parent 1780316118
commit 6329ddbbde
No known key found for this signature in database
GPG key ID: 4AEE18F83AFDEB23
1 changed files with 23 additions and 3 deletions
Download patch file Download diff file Expand all files Collapse all files

26
x-pack/plugins/data_visualizer/public/application/index_data_visualizer/search_strategy/requests/get_document_stats.ts
View file

@ -121,7 +121,7 @@ export const getDocumentCountStats = async (
},
});
const getSearchParams = (aggregations: unknown) => ({
const getSearchParams = (aggregations: unknown, trackTotalHits = false) => ({
index,
body: {
query,
@ -133,13 +133,17 @@ export const getDocumentCountStats = async (
: {}),
...(isPopulatedObject(runtimeFieldMap) ? { runtime_mappings: runtimeFieldMap } : {}),
},
track_total_hits: false,
track_total_hits: trackTotalHits,
size: 0,
});
const firstResp = await search
.search(
{
params: getSearchParams(getAggsWithRandomSampling(initialDefaultProbability)),
params: getSearchParams(
getAggsWithRandomSampling(initialDefaultProbability),
// Track total hits if time field is not defined
timeFieldName === undefined
),
},
searchOptions
)
@ -152,6 +156,22 @@ export const getDocumentCountStats = async (
)}`
);
}
// If time field is not defined, no need to show the document count chart
// Just need to return the tracked total hits
if (timeFieldName === undefined) {
const trackedTotalHits =
typeof firstResp.rawResponse.hits.total === 'number'
? firstResp.rawResponse.hits.total
: firstResp.rawResponse.hits.total?.value;
return {
...result,
randomlySampled: false,
took: firstResp.rawResponse.took,
totalCount: trackedTotalHits ?? 0,
};
}
if (isDefined(probability)) {
return {
...result,