Hi Liu, Options hints could be a security concern since users can override settings. However, query hints specifically could affect performance. Since we have a config to disable Options hint, I'm suggesting we also have a config to disable Query hints.
On Wed, Aug 16, 2023 at 9:41 AM liu ron <ron9....@gmail.com> wrote: > Hi, > > Thanks for driving this proposal. > > Can you explain why you would need to disable query hints because of > security issues? I don't really understand why query hints affects > security. > > Best, > Ron > > Bonnie Arogyam Varghese <bvargh...@confluent.io.invalid> 于2023年8月16日周三 > 23:59写道： > > > Platform providers may want to disable hints completely for security > > reasons. > > > > Currently, there is a configuration to disable OPTIONS hint - > > > > > https://nightlies.apache.org/flink/flink-docs-master/docs/dev/table/config/#table-dynamic-table-options-enabled > > > > However, there is no configuration available to disable QUERY hints - > > > > > https://nightlies.apache.org/flink/flink-docs-release-1.17/docs/dev/table/sql/queries/hints/#query-hints > > > > The proposal is to add a new configuration: > > > > Name: table.query-options.enabled > > Description: Enable or disable the QUERY hint, if disabled, an > > exception would be thrown if any QUERY hints are specified > > Note: The default value will be set to true. > > >
