Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Document issue about "CREATE MATERIALIZED VIEW" #47

Open
congxuebin opened this issue Nov 1, 2023 · 4 comments
Open

Document issue about "CREATE MATERIALIZED VIEW" #47

congxuebin opened this issue Nov 1, 2023 · 4 comments
Assignees

Comments

@congxuebin
Copy link

congxuebin commented Nov 1, 2023

https://github.com/cloudberrydb/cloudberrydb-site/blob/cbdb-doc-validation/docs/sql-stmts/sql-stmt-create-materialized-view.md


CREATE MATERIALIZED VIEW [ IF NOT EXISTS ] <table_name>
[ (<column_name> [, ...] ) ]
[ USING ]
[ WITH ( <storage_parameter> [= ] [, ... ] ) ]
[ TABLESPACE <tablespace_name> ]
AS
[ WITH [ NO ] DATA ]
[DISTRIBUTED {| BY [], [ ... ] | RANDOMLY | REPLICATED }]


DISTRIBUTED clause is not correct. Missing '(' and '|' should not be in front of 'BY'.

The following is an actual example tried on CBDB.

gpadmin=# CREATE MATERIALIZED VIEW mt11 (mc1, mc2, mc3) as select c1,c2,c3 from t1 distributed by (mc1 int4_ops,mc2 text_ops);
SELECT 4

Copy link

github-actions bot commented Nov 1, 2023

Hey, @congxuebin welcome!🎊 Thanks for taking the time to point this out.🙌

@TomShawn
Copy link
Contributor

TomShawn commented Nov 1, 2023

@congxuebin Hi, could you please create a pull request to correct this document? This document is in the cbdb-doc-validation branch, which is a temporary branch to store documents that have not been validated yet. These documents are in preparation and not published on the CBDB site yet. If you find anything wrong on these documents during your validation, you are welcome to create PR to correct them. Thanks~

congxuebin added a commit to congxuebin/cloudberrydb-site that referenced this issue Nov 1, 2023
@congxuebin
Copy link
Author

congxuebin commented Nov 1, 2023

@TomShawn The following is PR with my proposed change. It appears it did not pass certain check. Could you please help?

#48

@TomShawn
Copy link
Contributor

TomShawn commented Nov 2, 2023

@TomShawn The following is PR with my proposed change. It appears it did not pass certain check. Could you please help?

#48

Thanks! I'll take a look.

tuhaihe pushed a commit to congxuebin/cloudberrydb-site that referenced this issue Jul 18, 2024
tuhaihe pushed a commit that referenced this issue Jul 18, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants