Skip to content

Commit 734a8a1

Browse files
reshkemy-ship-it
authored andcommitted
Update index stats test to be in sync with 3d351d9
Change vacuum row progress expected output Fix vacuum_progress_column exp output Also some little refactoring done around src/backend/catalog/pg_attribute_encoding.c
1 parent 1591a3e commit 734a8a1

6 files changed

Lines changed: 18 additions & 15 deletions

File tree

src/backend/access/aocs/aocsam.c

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -134,15 +134,15 @@ open_all_datumstreamread_segfiles(AOCSScanDesc scan, AOCSFileSegInfo *segInfo)
134134
static void
135135
open_ds_write(Relation rel, DatumStreamWrite **ds, TupleDesc relationTupleDesc, bool checksum)
136136
{
137-
int nvp = relationTupleDesc->natts;
137+
int natts = RelationGetNumberOfAttributes(rel);
138138
StdRdOptions **opts = RelationGetAttributeOptions(rel);
139139
RelFileNodeBackend rnode;
140140

141141
rnode.node = rel->rd_node;
142142
rnode.backend = rel->rd_backend;
143143

144144
/* open datum streams. It will open segment file underneath */
145-
for (int i = 0; i < nvp; ++i)
145+
for (int i = 0; i < natts; ++i)
146146
{
147147
Form_pg_attribute attr = TupleDescAttr(relationTupleDesc, i);
148148
char *ct;

src/backend/catalog/pg_attribute_encoding.c

Lines changed: 6 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -225,13 +225,16 @@ RelationGetAttributeOptions(Relation rel)
225225
Datum *dats;
226226
StdRdOptions **opts;
227227
int i;
228+
int natts;
229+
230+
natts = RelationGetNumberOfAttributes(rel);
228231

229-
opts = palloc0(RelationGetNumberOfAttributes(rel) * sizeof(StdRdOptions *));
232+
opts = palloc0(natts * sizeof(StdRdOptions *));
230233

231234
dats = get_rel_attoptions(RelationGetRelid(rel),
232-
RelationGetNumberOfAttributes(rel));
235+
natts);
233236

234-
for (i = 0; i < RelationGetNumberOfAttributes(rel); i++)
237+
for (i = 0; i < natts; i++)
235238
{
236239
if (DatumGetPointer(dats[i]) != NULL)
237240
{

src/test/isolation2/expected/vacuum_progress_column.out

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -49,7 +49,7 @@ DELETE 50000
4949
SELECT relpages, reltuples, relallvisible FROM pg_class where relname = 'vacuum_progress_ao_column';
5050
relpages | reltuples | relallvisible
5151
----------+-----------+---------------
52-
0 | 0 | 0
52+
0 | -1 | 0
5353
(1 row)
5454
SELECT n_live_tup, n_dead_tup, last_vacuum, vacuum_count FROM pg_stat_all_tables WHERE relname = 'vacuum_progress_ao_column';
5555
n_live_tup | n_dead_tup | last_vacuum | vacuum_count

src/test/isolation2/expected/vacuum_progress_row.out

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -49,7 +49,7 @@ DELETE 50000
4949
SELECT relpages, reltuples, relallvisible FROM pg_class where relname = 'vacuum_progress_ao_row';
5050
relpages | reltuples | relallvisible
5151
----------+-----------+---------------
52-
0 | 0 | 0
52+
0 | -1 | 0
5353
(1 row)
5454
SELECT n_live_tup, n_dead_tup, last_vacuum, vacuum_count FROM pg_stat_all_tables WHERE relname = 'vacuum_progress_ao_row';
5555
n_live_tup | n_dead_tup | last_vacuum | vacuum_count

src/test/regress/expected/uao_compaction/index_stats.out

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -59,9 +59,9 @@ vacuum mytab2;
5959
SELECT gp_segment_id, relname, reltuples FROM gp_dist_random('pg_class') WHERE relname = 'mytab2_int_idx1';
6060
gp_segment_id | relname | reltuples
6161
---------------+-----------------+-----------
62-
0 | mytab2_int_idx1 | 0
63-
1 | mytab2_int_idx1 | 0
64-
2 | mytab2_int_idx1 | 0
62+
0 | mytab2_int_idx1 | -1
63+
1 | mytab2_int_idx1 | -1
64+
2 | mytab2_int_idx1 | -1
6565
(3 rows)
6666

6767
-- second vacuum update index stat with table stat
@@ -93,7 +93,7 @@ insert into mytab3 values(1,'aa',1001,101),(2,'bb',1002,102);
9393
select reltuples from pg_class where relname='mytab3';
9494
reltuples
9595
-----------
96-
0
96+
-1
9797
(1 row)
9898

9999
-- inspect the state of the stats on segments

src/test/regress/expected/uaocs_compaction/index_stats.out

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -64,9 +64,9 @@ vacuum uaocs_index_stats2;
6464
SELECT gp_segment_id, relname, reltuples FROM gp_dist_random('pg_class') WHERE relname = 'uaocs_index_stats2_int_idx1';
6565
gp_segment_id | relname | reltuples
6666
---------------+-----------------------------+-----------
67-
0 | uaocs_index_stats2_int_idx1 | 0
68-
1 | uaocs_index_stats2_int_idx1 | 0
69-
2 | uaocs_index_stats2_int_idx1 | 0
67+
0 | uaocs_index_stats2_int_idx1 | -1
68+
1 | uaocs_index_stats2_int_idx1 | -1
69+
2 | uaocs_index_stats2_int_idx1 | -1
7070
(3 rows)
7171

7272
-- second vacuum update index stat with table stat
@@ -98,7 +98,7 @@ insert into uaocs_index_stats3 values(1,'aa',1001,101),(2,'bb',1002,102);
9898
select reltuples from pg_class where relname='uaocs_index_stats3';
9999
reltuples
100100
-----------
101-
0
101+
-1
102102
(1 row)
103103

104104
-- inspect the state of the stats on segments

0 commit comments

Comments
 (0)