-
Notifications
You must be signed in to change notification settings - Fork 8.1k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
fixes error with postFlightRequest #16643
Conversation
💔 Build Failed |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
To reproduce the actual issue - since it's not documented in the original issue - you should add a new aggregation (but don't select the aggregation type yet) and then switch the time range. |
34784b6
to
f57cefe
Compare
💔 Build Failed |
f57cefe
to
e100bc1
Compare
jenkins, test this |
💚 Build Succeeded |
# Conflicts: # src/ui/public/vis/request_handlers/courier.js
I think it'd be good to create a test that would have caught this issue (according to the ticket, looks like a user was unable to add sub aggregations). I suspect the creation of nestedSearchSource is not necessary (I'm trying to track down the various flows of search source inheritance aInd creation, which is how I found it). A test that would cover this part of the code could also help find an issue if removing nestedSearchSource breaks anything. |
resolves #15912