Location via proxy:   [ UP ]  
[Report a bug]   [Manage cookies]                
Remove duplicate code in brin_memtuple_initialize
authorTomas Vondra <tomas.vondra@postgresql.org>
Wed, 11 Nov 2020 17:37:36 +0000 (18:37 +0100)
committerTomas Vondra <tomas.vondra@postgresql.org>
Wed, 11 Nov 2020 17:59:41 +0000 (18:59 +0100)
Commit 8bf74967dab moved some of the code from brin_new_memtuple to
brin_memtuple_initialize, but this resulted in some of the code being
duplicate. Fix by removing the duplicate lines and backpatch to 10.

Author: Tomas Vondra
Backpatch-through: 10
Discussion: https://postgr.es/m/5eb50c97-9a8e-b691-8c40-1b2a55611c4c%40enterprisedb.com

src/backend/access/brin/brin_tuple.c

index 5a8174a2513b5a2342c0c0e98243593fc3b688bf..00146940c0fda8e4145997f02fd2664604fa438b 100644 (file)
@@ -482,9 +482,6 @@ brin_memtuple_initialize(BrinMemTuple *dtuple, BrinDesc *brdesc)
                 sizeof(BrinValues) * brdesc->bd_tupdesc->natts);
    for (i = 0; i < brdesc->bd_tupdesc->natts; i++)
    {
-       dtuple->bt_columns[i].bv_allnulls = true;
-       dtuple->bt_columns[i].bv_hasnulls = false;
-
        dtuple->bt_columns[i].bv_attno = i + 1;
        dtuple->bt_columns[i].bv_allnulls = true;
        dtuple->bt_columns[i].bv_hasnulls = false;