postgres.git / summary / log / commit / refs

commit    913a03ec29abcb6c20e5941e5f3ac0578619cd3e
Author:   Tom Lane <tgl@sss.pgh.pa.us>
Date:     Tue Jan 04 21:52:51 2022 +0000
First released in: 15.0

    Remove redundant initialization of BrinMemTuple.
    
    brin_new_memtuple already did this, so there's no need
    for initialize_brin_buildstate to do it again.
    
    Richard Guo, reviewed by Bharath Rupireddy
    
    Discussion: https://postgr.es/m/CAMbWs4-kYYpKNOdiWtsCZ3jbkFFj4nhOVH22JH7dsrMYX=aGjg@mail.gmail.com


 src/backend/access/brin/brin.c | 2 --
 1 file changed, 2 deletions(-)

diff --git a/src/backend/access/brin/brin.c b/src/backend/access/brin/brin.c
index f521bb96356..c7d05ce68ba 100644
--- a/src/backend/access/brin/brin.c
+++ b/src/backend/access/brin/brin.c
@@ -1262,8 +1262,6 @@ initialize_brin_buildstate(Relation idxRel, BrinRevmap *revmap,
 	state->bs_bdesc = brin_build_desc(idxRel);
 	state->bs_dtuple = brin_new_memtuple(state->bs_bdesc);
 
-	brin_memtuple_initialize(state->bs_dtuple, state->bs_bdesc);
-
 	return state;
 }
 


[parent: 67a8cb5cbfff]