Github user clarkyzl commented on a diff in the pull request:
https://github.com/apache/flink/pull/3623#discussion_r110086591
--- Diff:
flink-libraries/flink-table/src/main/scala/org/apache/flink/table/functions/utils/TableSqlFunction.scala
---
@@ -53,22 +54,9 @@ class TableSqlFunction(
paramTypes,
functionImpl) {
- /**
- * Get the user-defined table function.
- */
- def getTableFunction = udtf
-
- /**
- * Get the type information of the table returned by the table function.
- */
- def getRowTypeInfo = rowTypeInfo
-
- /**
- * Get additional mapping information if the returned table type is a
POJO
- * (POJO types have no deterministic field order).
- */
- def getPojoFieldMapping = functionImpl.fieldIndexes
--- End diff --
Yes. POJOs are still supported. The pojo field mappings are re-calculated
by the dynamic schema. You can see in `DataSetCorrelate` and
`DataStreamCorrelate`. Also all the tests of POJOs still pass.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---
