dawidwys commented on code in PR #23223:
URL: https://github.com/apache/flink/pull/23223#discussion_r1342612321
##########
docs/layouts/shortcodes/generated/table_config_configuration.html:
##########
@@ -62,6 +62,12 @@
<td>String</td>
<td>The local time zone defines current session time zone id. It
is used when converting to/from <code>TIMESTAMP WITH LOCAL TIME
ZONE</code>. Internally, timestamps with local time zone are always
represented in the UTC time zone. However, when converting to data types that
don't include a time zone (e.g. TIMESTAMP, TIME, or simply STRING), the session
time zone is used during conversion. The input of option is either a full name
such as "America/Los_Angeles", or a custom timezone id such as "GMT-08:00".</td>
</tr>
+ <tr>
+ <td><h5>table.optimizer.query-hints.enabled</h5><br> <span
class="label label-primary">Batch</span> <span class="label
label-primary">Streaming</span></td>
Review Comment:
@bvarghese1 Let's tentatively revert to
`table.optimizer.query-hints.enabled`, as I agree with you it is more
consistent with other options. After that I'll merge the PR. Thanks for working
on that!
--
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
