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

Remove unnecessary pre-fastpath code (#7262) 

This code was here because we first implemented
`fast path planner` via
[#2606](https://github.com/citusdata/citus/pull/2606)
and then later `deferred pruning`
[#3369](https://github.com/citusdata/citus/pull/3369)
So, for some years, this code was useful.
pull/7261/head
Naisila Puka 2023-10-23 13:01:48 +03:00 committed by GitHub
parent 2d1444188c
commit 1fe16fa746
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
1 changed files with 3 additions and 19 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

18
src/backend/distributed/planner/multi_router_planner.c
View File

@ -2324,28 +2324,12 @@ PlanRouterQuery(Query *originalQuery,
TargetShardIntervalForFastPathQuery(originalQuery, &isMultiShardQuery, TargetShardIntervalForFastPathQuery(originalQuery, &isMultiShardQuery,
distributionKeyValue, distributionKeyValue,
partitionValueConst); partitionValueConst);
Assert(!isMultiShardQuery);
/*
* This could only happen when there is a parameter on the distribution key.
* We defer error here, later the planner is forced to use a generic plan
* by assigning arbitrarily high cost to the plan.
*/
if (UpdateOrDeleteOrMergeQuery(originalQuery) && isMultiShardQuery)
{
planningError = DeferredError(ERRCODE_FEATURE_NOT_SUPPORTED,
"Router planner cannot handle multi-shard "
"modify queries", NULL, NULL);
return planningError;
}
*prunedShardIntervalListList = shardIntervalList; *prunedShardIntervalListList = shardIntervalList;
if (!isMultiShardQuery)
{
ereport(DEBUG2, (errmsg("Distributed planning for a fast-path router " ereport(DEBUG2, (errmsg("Distributed planning for a fast-path router "
"query"))); "query")));
} }
}
else else
{ {
*prunedShardIntervalListList = *prunedShardIntervalListList =