Hi, Here is a comment for ExecInitPartitionInfo:
296 * ExecInitPartitionInfo 297 * Initialize ResultRelInfo and other information for a partition if not 298 * already done I think we should remove the words "if not already done" from that comment because 1) that function is called if the partition wasn't already initialized and 2) that function assumes that. Attached is a small patch for removing the words. Best regards, Etsuro Fujita
diff --git a/src/backend/executor/execPartition.c b/src/backend/executor/execPartition.c index a2f6b29..4c9f71f 100644 --- a/src/backend/executor/execPartition.c +++ b/src/backend/executor/execPartition.c @@ -294,8 +294,7 @@ ExecFindPartition(ResultRelInfo *resultRelInfo, PartitionDispatch *pd, /* * ExecInitPartitionInfo - * Initialize ResultRelInfo and other information for a partition if not - * already done + * Initialize ResultRelInfo and other information for a partition * * Returns the ResultRelInfo */
