Skip to content

Commit a3fc90d

Browse files
reshkeavamingli
authored andcommitted
resolve cherry-pick issues
1 parent 2f80123 commit a3fc90d

File tree

6 files changed

+7
-26
lines changed

6 files changed

+7
-26
lines changed

src/bin/pg_dump/pg_dump.c

+2-21
Original file line numberDiff line numberDiff line change
@@ -5336,15 +5336,6 @@ create_ao_relname(char *dst, size_t len, const char *prefix, Oid auxoid)
53365336
fatal("create_ao_relname: destination buffer is too short");
53375337
}
53385338

5339-
static void
5340-
create_ao_idxname(char *dst, size_t len, const char *prefix, Oid auxoid)
5341-
{
5342-
size_t actual = snprintf(dst, len, "%s_%u_index", prefix, auxoid);
5343-
5344-
if (actual >= len)
5345-
fatal("create_ao_idxname: destination buffer is too short");
5346-
}
5347-
53485339
/*
53495340
* GPDB: the implementation of this function has moved below, to
53505341
* binary_upgrade_set_pg_class_oids_impl(), so that we can handle some of the
@@ -5389,9 +5380,7 @@ binary_upgrade_set_pg_class_oids_impl(Archive *fout,
53895380
Oid pg_class_bmidxoid;
53905381
Oid ao_segrelid = InvalidOid;
53915382
Oid ao_blkdirrelid = InvalidOid;
5392-
Oid ao_blkdiridxid = InvalidOid;
53935383
Oid ao_visimaprelid = InvalidOid;
5394-
Oid ao_visimapidxid = InvalidOid;
53955384
bool ao_columnstore = false;
53965385
char pg_class_relkind;
53975386

@@ -5402,8 +5391,8 @@ binary_upgrade_set_pg_class_oids_impl(Archive *fout,
54025391
" i.indexrelid, ti.relname AS tidx_relname, "
54035392
" bi.oid AS bmoid, bidx.oid AS bmidxoid, "
54045393
" pgao.segrelid, pgao.columnstore, "
5405-
" pgao.blkdirrelid, pgao.blkdiridxid, "
5406-
" pgao.visimaprelid, pgao.visimapidxid "
5394+
" pgao.blkdirrelid, "
5395+
" pgao.visimaprelid "
54075396
"FROM pg_catalog.pg_class c "
54085397
"LEFT JOIN pg_catalog.pg_class t ON (c.reltoastrelid = t.oid) "
54095398
"LEFT JOIN pg_catalog.pg_index i ON (c.reltoastrelid = i.indrelid AND i.indisvalid) "
@@ -5442,9 +5431,7 @@ binary_upgrade_set_pg_class_oids_impl(Archive *fout,
54425431
ao_segrelid = atooid(PQgetvalue(upgrade_res, 0, PQfnumber(upgrade_res, "segrelid")));
54435432
ao_columnstore = (PQgetvalue(upgrade_res, 0, PQfnumber(upgrade_res, "columnstore"))[0] == 't');
54445433
ao_blkdirrelid = atooid(PQgetvalue(upgrade_res, 0, PQfnumber(upgrade_res, "blkdirrelid")));
5445-
ao_blkdiridxid = atooid(PQgetvalue(upgrade_res, 0, PQfnumber(upgrade_res, "blkdiridxid")));
54465434
ao_visimaprelid = atooid(PQgetvalue(upgrade_res, 0, PQfnumber(upgrade_res, "visimaprelid")));
5447-
ao_visimapidxid = atooid(PQgetvalue(upgrade_res, 0, PQfnumber(upgrade_res, "visimapidxid")));
54485435
}
54495436

54505437

@@ -5526,17 +5513,11 @@ binary_upgrade_set_pg_class_oids_impl(Archive *fout,
55265513
create_ao_relname(ao_relname, sizeof(ao_relname), "pg_aoblkdir", pg_class_oid);
55275514
binary_upgrade_set_pg_class_oids_for_ao(fout, upgrade_buffer, ao_blkdirrelid, false, ao_relname);
55285515
binary_upgrade_set_type_oids_for_ao(fout, upgrade_buffer, ao_blkdirrelid, ao_relname);
5529-
5530-
create_ao_idxname(ao_relname, sizeof(ao_relname), "pg_aoblkdir", pg_class_oid);
5531-
binary_upgrade_set_pg_class_oids_for_ao(fout, upgrade_buffer, ao_blkdiridxid, true, ao_relname);
55325516
}
55335517

55345518
create_ao_relname(ao_relname, sizeof(ao_relname), "pg_aovisimap", pg_class_oid);
55355519
binary_upgrade_set_pg_class_oids_for_ao(fout, upgrade_buffer, ao_visimaprelid, false, ao_relname);
55365520
binary_upgrade_set_type_oids_for_ao(fout, upgrade_buffer, ao_visimaprelid, ao_relname);
5537-
5538-
create_ao_idxname(ao_relname, sizeof(ao_relname), "pg_aovisimap", pg_class_oid);
5539-
binary_upgrade_set_pg_class_oids_for_ao(fout, upgrade_buffer, ao_visimapidxid, true, ao_relname);
55405521
}
55415522

55425523
PQclear(upgrade_res);

src/test/isolation2/expected/ao_index_build_progress.out

+1-1
Original file line numberDiff line numberDiff line change
@@ -110,7 +110,7 @@ SELECT gp_wait_until_triggered_fault('AppendOnlyStorageRead_ReadNextBlock_succes
110110
1U: SELECT command, phase, ((pg_stat_file(pg_relation_filepath('aoco_index_build_progress') || '.' || 129)).size + (current_setting('block_size')::int - 1)) / current_setting('block_size')::int AS col_j_blocks, blocks_total AS blocks_total_reported, blocks_done AS blocks_done_reported FROM pg_stat_progress_create_index WHERE relid = 'aoco_index_build_progress'::regclass;
111111
command | phase | col_j_blocks | blocks_total_reported | blocks_done_reported
112112
--------------+--------------------------------+--------------+-----------------------+----------------------
113-
CREATE INDEX | building index: scanning table | 8 | 20 | 4
113+
CREATE INDEX | building index: scanning table | 8 | 20 | 3
114114
(1 row)
115115

116116
SELECT gp_inject_fault('AppendOnlyStorageRead_ReadNextBlock_success', 'reset', dbid) FROM gp_segment_configuration WHERE content = 1 AND role = 'p';

src/test/regress/output/uao_ddl/alter_ao_part_tables_splitpartition.source

+1-1
Original file line numberDiff line numberDiff line change
@@ -15,7 +15,7 @@ pg_class c join pg_namespace n on c.relnamespace = n.oid and
1515
c.relname like 'sto_alt_uao_part_splitpartition%' and
1616
n.nspname = 'alter_ao_part_tables_splitpartition_@amname@');
1717
visimapcount
18-
-------------
18+
--------------
1919
3
2020
(1 row)
2121

src/test/regress/output/uao_ddl/analyze_ao_table_every_dml.source

+1-1
Original file line numberDiff line numberDiff line change
@@ -28,7 +28,7 @@ select relname, reltuples from pg_class where oid='sto_uao_city_analyze_everydml
2828
SELECT count(i.indexrelid) = 1 AS VisimapPresent FROM pg_appendonly a, pg_index i WHERE a.visimaprelid = i.indrelid AND
2929
a.relid='sto_uao_city_analyze_everydml'::regclass;
3030
visimappresent
31-
------------
31+
----------------
3232
t
3333
(1 row)
3434

src/test/regress/output/uao_ddl/create_ao_table_500cols.source

+1-1
Original file line numberDiff line numberDiff line change
@@ -113,7 +113,7 @@ a498 inet, a499 time, a500 text)
113113
SELECT count(i.indexrelid) = 1 AS VisimapPresent FROM pg_appendonly a, pg_index i WHERE a.visimaprelid = i.indrelid AND
114114
a.relid='sto_uao_500cols'::regclass;
115115
visimappresent
116-
------------
116+
----------------
117117
t
118118
(1 row)
119119

src/test/regress/output/uao_ddl/create_ao_tables.source

+1-1
Original file line numberDiff line numberDiff line change
@@ -292,7 +292,7 @@ set gp_select_invisible=false;
292292
SELECT count(i.indexrelid) = 1 AS VisimapPresent FROM pg_appendonly a, pg_index i WHERE a.visimaprelid = i.indrelid AND
293293
a.relid='sto_uao_9'::regclass;
294294
visimappresent
295-
------------
295+
----------------
296296
t
297297
(1 row)
298298

0 commit comments

Comments
 (0)