external
/
citus
mirror of https://github.com/citusdata/citus.git
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

Get rid of an unnecessary parameter 

targetPoolSize parameter for ExecuteUtilityTaskListWithoutResults
becomes obsolete, just remove it.
release-8.3
Onder Kalaci 2019-08-06 12:23:03 +02:00
parent dd1a838e58
commit 4ee803bf41
5 changed files with 7 additions and 9 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

5
src/backend/distributed/commands/utility_hook.c
View File

@ -553,7 +553,6 @@ static void
ExecuteDistributedDDLJob(DDLJob *ddlJob)
{
bool shouldSyncMetadata = ShouldSyncTableMetadata(ddlJob->targetRelationId);
int targetPoolSize = MaxAdaptiveExecutorPoolSize;
EnsureCoordinator();
EnsurePartitionTableNotReplicated(ddlJob->targetRelationId);
@ -579,7 +578,7 @@ ExecuteDistributedDDLJob(DDLJob *ddlJob)
}
/* use adaptive executor when enabled */
ExecuteUtilityTaskListWithoutResults(ddlJob->taskList, targetPoolSize);
ExecuteUtilityTaskListWithoutResults(ddlJob->taskList);
}
else
{
@ -591,7 +590,7 @@ ExecuteDistributedDDLJob(DDLJob *ddlJob)
PG_TRY();
{
/* use adaptive executor when enabled */
ExecuteUtilityTaskListWithoutResults(ddlJob->taskList, targetPoolSize);
ExecuteUtilityTaskListWithoutResults(ddlJob->taskList);
if (shouldSyncMetadata)
{

3
src/backend/distributed/commands/vacuum.c
View File

@ -74,7 +74,6 @@ ProcessVacuumStmt(VacuumStmt *vacuumStmt, const char *vacuumCommand)
{
List *vacuumColumnList = NIL;
List *taskList = NIL;
int targetPoolSize = MaxAdaptiveExecutorPoolSize;
/*
* VACUUM commands cannot run inside a transaction block, so we use
@ -94,7 +93,7 @@ ProcessVacuumStmt(VacuumStmt *vacuumStmt, const char *vacuumCommand)
taskList = VacuumTaskList(relationId, vacuumStmt->options, vacuumColumnList);
/* use adaptive executor when enabled */
ExecuteUtilityTaskListWithoutResults(taskList, targetPoolSize);
ExecuteUtilityTaskListWithoutResults(taskList);
executedVacuumCount++;
}
relationIndex++;

4
src/backend/distributed/executor/adaptive_executor.c
View File

@ -657,11 +657,11 @@ AdaptiveExecutor(CustomScanState *node)
* through router executor.
*/
void
ExecuteUtilityTaskListWithoutResults(List *taskList, int targetPoolSize)
ExecuteUtilityTaskListWithoutResults(List *taskList)
{
if (TaskExecutorType == MULTI_EXECUTOR_ADAPTIVE)
{
ExecuteTaskList(CMD_UTILITY, taskList, targetPoolSize);
ExecuteTaskList(CMD_UTILITY, taskList, MaxAdaptiveExecutorPoolSize);
}
else
{

2
src/backend/distributed/master/master_truncate.c
View File

@ -77,7 +77,7 @@ citus_truncate_trigger(PG_FUNCTION_ARGS)
{
List *taskList = TruncateTaskList(relationId);
ExecuteUtilityTaskListWithoutResults(taskList, MaxAdaptiveExecutorPoolSize);
ExecuteUtilityTaskListWithoutResults(taskList);
}
PG_RETURN_DATUM(PointerGetDatum(NULL));

2
src/include/distributed/multi_executor.h
View File

@ -43,7 +43,7 @@ extern uint64 ExecuteTaskListExtended(CmdType operation, List *taskList,
Tuplestorestate *tupleStore,
bool hasReturning, int targetPoolSize);
extern void ExecuteUtilityTaskListWithoutResults(List *taskList, int targetPoolSize);
extern void ExecuteUtilityTaskListWithoutResults(List *taskList);
extern uint64 ExecuteTaskList(CmdType operation, List *taskList, int
targetPoolSize);
extern TupleTableSlot * CitusExecScan(CustomScanState *node);