Re: May cause infinite loop when initializing rel-cache contains partitioned table

Hi,

On 2017/05/05 17:26, 甄明洋 wrote:
> The function of initializing rel-cache is RelationCacheInitializePhase3 and in src/backend/utils/cache/relcache.c file.
> When initializing the partition description information, we forget to check if partition key or descriptor is NULL.
> Therefore, after the loop restarts, the partition information will be initialized again, resulting in an infinite loop.
> Code:
> /*
> * Reload partition key and descriptor for a partitioned table.
> */
> if (relation->rd_rel->relkind == RELKIND_PARTITIONED_TABLE)
> {
> RelationBuildPartitionKey(relation);
> Assert(relation->rd_partkey != NULL);
>
>
> RelationBuildPartitionDesc(relation);
> Assert(relation->rd_partdesc != NULL);
>
>
> restart = true;
> }

Thanks for bringing it to the notice. The above code should follow what's
done for other fields that are initialized by
RelationCacheInitializePhase3(). Although, since none of the entries in
the relcache init file are partitioned tables, infinite loop won't occur
in practice, but it's a good idea to fix the code anyway.

Attached patch does that.

Thanks,
Amit