snuyanzin commented on code in PR #25186:
URL: https://github.com/apache/flink/pull/25186#discussion_r1711563695
##########
flink-table/flink-table-planner/src/main/java/org/apache/flink/table/planner/plan/schema/CatalogSourceTable.java:
##########
@@ -119,7 +119,7 @@ public RelNode toRel(ToRelContext toRelContext) {
!schemaTable.isStreamingMode(),
context.getTableConfig(),
relBuilder,
- schemaTable.getContextResolvedTable(),
+ catalogTable,
Review Comment:
>So the resolved table does not include the hints?
in fact it didn't include hints before line 108 in this class
https://github.com/apache/flink/blob/7f0e0e81bac2cc512d0669b25995ca3b13e262f9/flink-table/flink-table-planner/src/main/java/org/apache/flink/table/planner/plan/schema/CatalogSourceTable.java#L108-L111
where then hints (including option hint) are going to be merged into
`catalogTable`, however for some reason it was not used here
--
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]
