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
Смирнов Андрей ИТМО КТ Stage 5 #181
Смирнов Андрей ИТМО КТ Stage 5 #181
Changes from 49 commits
33dec72
4ed27a3
215a47c
b4630db
62ee00a
474f3f0
2cd4f41
ea708bc
6197f2f
11f2f3f
8ba7877
508c025
a10c6db
a0aa716
dd51faa
a3d2c05
b4e208a
cf7c014
d816a3a
dcdc8d2
fabb285
9b5f96d
3673f33
506da2d
7da5b8e
1deba38
f916e7e
0023be5
fce0383
2d05f98
3895f6e
f4b18fd
31eef05
7a7bdf5
8b125f3
0fa4876
906f9e3
8fd039f
3ab4bca
a5f3699
fe005fd
359ca1b
745c99f
b103ff5
7a2d010
33303de
7b503fa
9877d83
9007e40
fe3810b
60e8b9e
63f606d
4466032
File filter
Filter by extension
Conversations
Jump to
There are no files selected for viewing
Large diffs are not rendered by default.
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.
На каком пуле потоков выполняется агрегация ответов от реплик?
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.
На том же, что задается вот в этой строчке
this.executor = new MyExecutor(corePoolSize, availableProcessors);
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.
Так как в явном виде не меняется пул потоков (не передается другой executor), то будет выполняться на том же пуле потоков, что и completableResults, то есть на том же пуле, что и sendAsync, на потоках из httpClient.