Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
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
[8.x] Add Pre-8.x Enterprise Search Index Incompatibility Deprecations #210688
[8.x] Add Pre-8.x Enterprise Search Index Incompatibility Deprecations #210688
Changes from 9 commits
0404447
a3b0968
68b8e4d
de29f95
27fcc2a
8741e59
8ee37ce
1946c84
fa2811a
1421a3f
07221da
896a472
f226f0c
2a92eff
db47ab5
eed2217
1b3051b
97de4f9
File filter
Filter by extension
Conversations
Jump to
There are no files selected for viewing
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.
sorry I missed this in the original PR but these inner
awaits
should not be here.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.
Thanks @TattdCodeMonkey - Just to be sure (and for my own education too) - with the
awaits
there, basically, this will wait for each one, in sequence, to complete before creating a promise with all the completed tasks... when, what should be is that theawait Promise.all
call will itself wait for all the promises to be resolved before completing... correct? (i.e. without the individualawait
keywords for each of the async functions, those functions can essentially run in parallel... whereas as the moment, they will not... correct?)