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


Reply via email to