search

cloudberrydb / cloudberrydb-site

Website sources for Cloudberry Database
http://cloudberrydb.org/
Apache License 2.0
9 stars 12 forks source link

Document issue about "CREATE MATERIALIZED VIEW" #47

Open congxuebin opened 11 months ago

congxuebin commented 11 months ago

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 ] [ ( [, ...] ) ] [ USING ] [ WITH ( [= ] [, ... ] ) ] [ TABLESPACE ] 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

github-actions[bot] commented 11 months ago

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

TomShawn commented 11 months ago

@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 commented 11 months ago

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

https://github.com/cloudberrydb/cloudberrydb-site/pull/48

TomShawn commented 11 months ago

@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.