citusdata / citusdata/citus

Should we unify pre/post process DDL functions ?

Open
#5,839 4 comments 0 reactions 0 assignees View on GitHub
DDL
Dominant language
C
Stars
12.8k
Forks
794
Avg merge
2d 14h
Merged PRs (30d)
31

Description

Some examples:
* For some commands, we call EnsureCoordinator() in preprocess and do nothing else, and for some others, we call QualifyTreeNode and do nothing else.
* For some others, we do both or maybe neither of them.
* Even more, there are some DDLs that we don't have a preprocess function at all.

Consider if those are by design or are simply inconsistencies that we couldn't realize when extending distributed object infrastructure.

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.