Обсуждение: minor refactor on src/test/modules/test_ddl_deparse/sql/alter_table.sql

Поиск
Список
Период
Сортировка

minor refactor on src/test/modules/test_ddl_deparse/sql/alter_table.sql

От
jian he
Дата:
hi.

in src/test/modules/test_ddl_deparse/sql/alter_table.sql:
----------------------------
ALTER TABLE parent ADD CONSTRAINT a_pos CHECK (a > 0);
CREATE TABLE part (
    a int
) PARTITION BY RANGE (a);

CREATE TABLE part1 PARTITION OF part FOR VALUES FROM (1) to (100);
CREATE TABLE part2 (a int);
ALTER TABLE part ATTACH PARTITION part2 FOR VALUES FROM (101) to (200);
ALTER TABLE part DETACH PARTITION part2;
DROP TABLE part2;
ALTER TABLE part ADD PRIMARY KEY (a);
ALTER TABLE parent ALTER COLUMN a SET NOT NULL;

----
The test initially focuses on the "parent" table, then switches to the "part"
table, and goes back to the "parent" table.
This seems weird?  so I slightly adjusted the order to cover the "parent"
table first, followed by the "part" table.

src/test/modules/test_ddl_deparse/sql/alter_table.sql
don't have ALTER TABLE ALTER COLUMN SET EXPRESSION,
so I added a test on it.

Вложения

Re: minor refactor on src/test/modules/test_ddl_deparse/sql/alter_table.sql

От
Chao Li
Дата:


On Aug 25, 2025, at 08:00, jian he <jian.universality@gmail.com> wrote:

hi.

in src/test/modules/test_ddl_deparse/sql/alter_table.sql:

----
The test initially focuses on the "parent" table, then switches to the "part"
table, and goes back to the "parent" table.
This seems weird?  so I slightly adjusted the order to cover the "parent"
table first, followed by the "part" table.

I think this change makes the test more organized.


src/test/modules/test_ddl_deparse/sql/alter_table.sql
don't have ALTER TABLE ALTER COLUMN SET EXPRESSION,
so I added a test on it.
<v1-0001-refactor-test_ddl_deparse-sql-alter_table.sql.patch>


I confirm “make check” passed with this patch.

--
Chao Li (Evan)
HighGo Software Co., Ltd.
https://www.highgo.com/