citusdata / citusdata/citus

create_distributed_table_concurrently with unlogged tables should give better error message

Open
#6,374 1 comment 0 reactions 0 assignees View on GitHub
enhancement warm-up
Dominant language
C
Stars
12.8k
Forks
794
Avg merge
2d 14h
Merged PRs (30d)
31

Description

We should give the error much earlier:

```SQL
CREATE UNLOGGED TABLE unlogged_dist(a bigserial PRIMARY KEY);
SELECT create_distributed_table_concurrently('unlogged_dist', 'a');
ERROR: cannot add relation "unlogged_dist_102075" to publication
DETAIL: This operation is not supported for unlogged tables.
```

Related to https://github.com/citusdata/citus-enterprise/issues/394

Contributor guide

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.