Rationalize some ThreadPool Use in Snapshot Transport Actions (#54772) (#54782)

Removing a few spots where we clearly don't have to fork to the generic or management
pool since either we only interpret the current cluster state or fork-off directly to
some other pool in the transport action logic anyway.
This commit is contained in:
Armin Braun 2020-04-06 09:57:41 +02:00 committed by GitHub
parent 30b01fe00d
commit baff7bfa14
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
3 changed files with 3 additions and 5 deletions

View File

@ -60,7 +60,7 @@ public class TransportGetRepositoriesAction extends
@Override
protected String executor() {
return ThreadPool.Names.MANAGEMENT;
return ThreadPool.Names.SAME;
}
@Override

View File

@ -56,7 +56,7 @@ public class TransportVerifyRepositoryAction extends
@Override
protected String executor() {
return ThreadPool.Names.MANAGEMENT;
return ThreadPool.Names.SAME;
}
@Override

View File

@ -52,9 +52,7 @@ public class TransportCreateSnapshotAction extends TransportMasterNodeAction<Cre
@Override
protected String executor() {
// Using the generic instead of the snapshot threadpool here as the snapshot threadpool might be blocked on long running tasks
// which would block the request from getting an error response because of the ongoing task
return ThreadPool.Names.GENERIC;
return ThreadPool.Names.SAME;
}
@Override