Skip to content

2.23.1.0-b52

Summary:
This reverts commit 52f7e799e4b329cf886fb2799e26658e12f668a1.
The original commit disabled setting `p_retention_schema` parameter for `drop_partition_id` and `drop_partition_time` as `ALTER TABLE SET SCHEMA` syntax was not supported but the support has been added by commit 34f9741b74e148b53c6d78961cc9e3533656b6e8.

Jira: DB-12001

Test Plan:
jenkins: compile only

Manual Test:
```
yugabyte=# CREATE SCHEMA IF NOT EXISTS partman;
CREATE SCHEMA
yugabyte=# CREATE EXTENSION pg_partman WITH SCHEMA partman;
CREATE EXTENSION
yugabyte=# SELECT set_config('search_path','partman, public',false);
   set_config
-----------------
 partman, public
(1 row)

yugabyte=# CREATE SCHEMA partman_test;
CREATE SCHEMA
yugabyte=# CREATE TABLE partman_test.time_taptest_table (col1 int, col2 text default 'stuff', col3 timestamptz NOT NULL DEFAULT now()) PARTITION BY RANGE (col3);
CREATE TABLE
yugabyte=# SELECT create_parent('partman_test.time_taptest_table', 'col3', 'native', 'daily');
 create_parent
---------------
 t
(1 row)

yugabyte=# UPDATE partman.part_config SET retention = '2 days' where parent_table = 'partman_test.time_taptest_table';
UPDATE 1
yugabyte=# CREATE SCHEMA partman_retention_test;
CREATE SCHEMA
yugabyte=# select drop_partition_time('partman_test.time_taptest_table', p_retention_schema := 'partman_retention_test');
 drop_partition_time
---------------------
                   2
(1 row)

yugabyte=# \d partman_retention_test.
      Table "partman_retention_test.time_taptest_table_p2024_08_16"
 Column |           Type           | Collation | Nullable |    Default
--------+--------------------------+-----------+----------+---------------
 col1   | integer                  |           |          |
 col2   | text                     |           |          | 'stuff'::text
 col3   | timestamp with time zone |           | not null | now()

      Table "partman_retention_test.time_taptest_table_p2024_08_17"
 Column |           Type           | Collation | Nullable |    Default
--------+--------------------------+-----------+----------+---------------
 col1   | integer                  |           |          |
 col2   | text                     |           |          | 'stuff'::text
 col3   | timestamp with time zone |           | not null | now()
```

Reviewers: jason

Reviewed By: jason

Subscribers: yql

Differential Revision: https://phorge.dev.yugabyte.com/D37403
Assets 2
Loading