wuchong commented on a change in pull request #16887: URL: https://github.com/apache/flink/pull/16887#discussion_r691727770
########## File path: docs/content/docs/dev/table/sql/queries/hints.md ########## @@ -50,8 +50,7 @@ these options can be specified flexibly in per-table scope within each query. Thus it is very suitable to use for the ad-hoc queries in interactive terminal, for example, in the SQL-CLI, you can specify to ignore the parse error for a CSV source just by adding a dynamic option `/*+ OPTIONS('csv.ignore-parse-errors'='true') */`. -<b>Note:</b> Dynamic table options default is forbidden to use because it may change the semantics of the query. -You need to set the config option `table.dynamic-table-options.enabled` to be `true` explicitly (default is false), +<b>Note:</b> Dynamic table options default is allowed, i.e. set the config option `table.dynamic-table-options.enabled` to be `true`, Review comment: It is enabled by default, so I think we don't need this note now. -- 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: issues-unsubscr...@flink.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org