You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
For CCS requests, the phase_took response is retrieved from a single remote cluster SearchResponse. This is incorrect when there are 2+ remote clusters.
getMergedResponse should aggregate phase_took data from all remote cluster SearchResponse objects. We can use sum, max, or average to maintain the response structure. I initially thought sum is best but this will break the expectation of total phase_took times roughly equalling took time.
Additional Details
Plugins
Please list all plugins currently enabled.
Screenshots
If applicable, add screenshots to help explain your problem.
Host/Environment (please complete the following information):
OS: [e.g. iOS]
Version [e.g. 22]
Additional context
Add any other context about the problem here.
The text was updated successfully, but these errors were encountered:
Describe the bug
For CCS requests, the phase_took response is retrieved from a single remote cluster SearchResponse. This is incorrect when there are 2+ remote clusters.
See getMergedResponse function
Related component
Search
To Reproduce
Expected behavior
getMergedResponse
should aggregate phase_took data from all remote cluster SearchResponse objects. We can use sum, max, or average to maintain the response structure. I initially thought sum is best but this will break the expectation of totalphase_took
times roughly equallingtook
time.Additional Details
Plugins
Please list all plugins currently enabled.
Screenshots
If applicable, add screenshots to help explain your problem.
Host/Environment (please complete the following information):
Additional context
Add any other context about the problem here.
The text was updated successfully, but these errors were encountered: