postgres.git / summary / log / commit / refs

commit    48de3bb1a50d3c53f50d827a05deee5adcbb936e
Author:   Etsuro Fujita <efujita@postgresql.org>
Date:     Mon Oct 21 08:30:01 2019 +0000
First released in: 13.0, 12.1

    Update obsolete comment.
    
    Commit b52b7dc25, which moved code creating PartitionBoundInfo in
    RelationBuildPartitionDesc() in partcache.c (relocated to partdesc.c
    afterwards) to partbounds.c, should have updated this, but didn't.
    
    Author: Etsuro Fujita
    Reviewed-by: Alvaro Herrera
    Backpatch-through: 12
    Discussion: https://postgr.es/m/CAPmGK16Uxr%3DPatiGyaRwiQVLB7Y-GqbkK3AxRLVYzU0Czv%3DsEw%40mail.gmail.com


 src/backend/partitioning/partbounds.c | 5 ++---
 1 file changed, 2 insertions(+), 3 deletions(-)

diff --git a/src/backend/partitioning/partbounds.c b/src/backend/partitioning/partbounds.c
index 90036b42dc6..f205cd435e1 100644
--- a/src/backend/partitioning/partbounds.c
+++ b/src/backend/partitioning/partbounds.c
@@ -360,9 +360,8 @@ create_list_bounds(PartitionBoundSpec **boundspecs, int nparts,
 			else
 			{
 				/*
-				 * Never put a null into the values array, flag instead for
-				 * the code further down below where we construct the actual
-				 * relcache struct.
+				 * Never put a null into the values array; save the index of
+				 * the partition that stores nulls, instead.
 				 */
 				if (null_index != -1)
 					elog(ERROR, "found null more than once");


[parent: 62f4dd3796da]