tuhaihe commented on code in PR #1030:
URL: https://github.com/apache/cloudberry/pull/1030#discussion_r2026004731
##########
src/test/regress/expected/aqumv.out:
##########
@@ -3173,31 +3173,31 @@ abort;
create table par(a int, b int, c int) partition by range(b)
subpartition by range(c) subpartition template (start (1) end (3) every
(1))
(start(1) end(3) every(1));
-NOTICE: Table doesn't have 'DISTRIBUTED BY' clause -- Using column named 'a'
as the Cloudberry Database data distribution key for this table.
+NOTICE: Table doesn't have 'DISTRIBUTED BY' clause -- Using column named 'a'
as the Apache Cloudberry data distribution key for this table.
HINT: The 'DISTRIBUTED BY' clause determines the distribution of data. Make
sure column(s) chosen are the optimal data distribution key to minimize skew.
Review Comment:
Hey, your suggestions would introduce significant changes to the source
files—about 3k+ lines, including NOTICEs. It might be better to make these
changes in a separate PR. What do you think? Hope to make this PR simpler.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: commits-unsubscr...@cloudberry.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscr...@cloudberry.apache.org
For additional commands, e-mail: commits-h...@cloudberry.apache.org
