davidradl commented on code in PR #25904:
URL: https://github.com/apache/flink/pull/25904#discussion_r1904223555
##########
docs/content.zh/docs/dev/table/materialized-table/statements.md:
##########
@@ -233,8 +233,8 @@ CREATE MATERIALIZED TABLE my_materialized_table_full
'format' = 'json',
'partition.fields.ds.date-formatter' = 'yyyy-MM-dd'
)
- FRESHNESS = INTERVAL '10' MINUTE
- REFRESH_MODE = 'FULL'
+ FRESHNESS = INTERVAL '1' HOUR
+ REFRESH_MODE = FULL
Review Comment:
@jiefei30 @lsyldliu I am unsure why we can remove `REFRESH_MODE =
CONTINUOUS` and the `REFRESH_MODE = FULL` in all the cases .
As the statement Materialized table syntax has REFRESH_MODE in square
brackets, this field is optional, so must have a default. I would assume we
only need to specify the value when it is not the default. We do not seem to
document the default for REFRESH_MODE.
Am I understanding this correctly?
--
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]
