Copilot commented on code in PR #3374:
URL: https://github.com/apache/doris-website/pull/3374#discussion_r2789729757
##########
docs/table-design/data-model/aggregate.md:
##########
@@ -45,17 +45,17 @@ When creating a table, the **AGGREGATE KEY** keyword can be
used to specify the
CREATE TABLE IF NOT EXISTS example_tbl_agg
(
user_id LARGEINT NOT NULL,
- load_dt DATE NOT NULL,
+ load_date DATE NOT NULL,
city VARCHAR(20),
last_visit_dt DATETIME REPLACE DEFAULT "1970-01-01 00:00:00",
cost BIGINT SUM DEFAULT "0",
max_dwell INT MAX DEFAULT "0",
)
-AGGREGATE KEY(user_id, load_dt, city)
+AGGREGATE KEY(user_id, load_date, city)
DISTRIBUTED BY HASH(user_id) BUCKETS 10;
```
-In the example above, a fact table for user information and access behavior is
defined, where `user_id`, `load_dt`, and `city` are used as Key columns for
aggregation. During data import, the Key columns are aggregated into one row,
and the Value columns are aggregated according to the specified aggregation
types.
+In the example above, a fact table for user information and access behavior is
defined, where `user_id`, `load_date`, and `city` are used as Key columns for
aggregation. During data import, the Key columns are aggregated into one row,
and the Value columns are aggregated according to the specified aggregation
types.
Review Comment:
This line ends with trailing whitespace after the final period, which can
cause noisy diffs and may fail markdown linting. Please remove the trailing
space at end-of-line.
```suggestion
In the example above, a fact table for user information and access behavior
is defined, where `user_id`, `load_date`, and `city` are used as Key columns
for aggregation. During data import, the Key columns are aggregated into one
row, and the Value columns are aggregated according to the specified
aggregation types.
```
##########
versioned_docs/version-3.x/table-design/data-model/aggregate.md:
##########
@@ -45,17 +45,17 @@ When creating a table, the **AGGREGATE KEY** keyword can be
used to specify the
CREATE TABLE IF NOT EXISTS example_tbl_agg
(
user_id LARGEINT NOT NULL,
- load_dt DATE NOT NULL,
+ load_date DATE NOT NULL,
city VARCHAR(20),
last_visit_dt DATETIME REPLACE DEFAULT "1970-01-01 00:00:00",
cost BIGINT SUM DEFAULT "0",
max_dwell INT MAX DEFAULT "0",
)
-AGGREGATE KEY(user_id, load_dt, city)
+AGGREGATE KEY(user_id, load_date, city)
DISTRIBUTED BY HASH(user_id) BUCKETS 10;
```
-In the example above, a fact table for user information and access behavior is
defined, where `user_id`, `load_dt`, and `city` are used as Key columns for
aggregation. During data import, the Key columns are aggregated into one row,
and the Value columns are aggregated according to the specified aggregation
types.
+In the example above, a fact table for user information and access behavior is
defined, where `user_id`, `load_date`, and `city` are used as Key columns for
aggregation. During data import, the Key columns are aggregated into one row,
and the Value columns are aggregated according to the specified aggregation
types.
Review Comment:
This line ends with trailing whitespace after the final period, which can
cause noisy diffs and may fail markdown linting. Please remove the trailing
space at end-of-line.
```suggestion
In the example above, a fact table for user information and access behavior
is defined, where `user_id`, `load_date`, and `city` are used as Key columns
for aggregation. During data import, the Key columns are aggregated into one
row, and the Value columns are aggregated according to the specified
aggregation types.
```
##########
versioned_docs/version-4.x/table-design/data-model/aggregate.md:
##########
@@ -45,17 +45,17 @@ When creating a table, the **AGGREGATE KEY** keyword can be
used to specify the
CREATE TABLE IF NOT EXISTS example_tbl_agg
(
user_id LARGEINT NOT NULL,
- load_dt DATE NOT NULL,
+ load_date DATE NOT NULL,
city VARCHAR(20),
last_visit_dt DATETIME REPLACE DEFAULT "1970-01-01 00:00:00",
cost BIGINT SUM DEFAULT "0",
max_dwell INT MAX DEFAULT "0",
)
-AGGREGATE KEY(user_id, load_dt, city)
+AGGREGATE KEY(user_id, load_date, city)
DISTRIBUTED BY HASH(user_id) BUCKETS 10;
```
-In the example above, a fact table for user information and access behavior is
defined, where `user_id`, `load_dt`, and `city` are used as Key columns for
aggregation. During data import, the Key columns are aggregated into one row,
and the Value columns are aggregated according to the specified aggregation
types.
+In the example above, a fact table for user information and access behavior is
defined, where `user_id`, `load_date`, and `city` are used as Key columns for
aggregation. During data import, the Key columns are aggregated into one row,
and the Value columns are aggregated according to the specified aggregation
types.
Review Comment:
This line ends with trailing whitespace after the final period, which can
cause noisy diffs and may fail markdown linting. Please remove the trailing
space at end-of-line.
```suggestion
In the example above, a fact table for user information and access behavior
is defined, where `user_id`, `load_date`, and `city` are used as Key columns
for aggregation. During data import, the Key columns are aggregated into one
row, and the Value columns are aggregated according to the specified
aggregation types.
```
--
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: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
