twalthr opened a new pull request #12900:
URL: https://github.com/apache/flink/pull/12900
## What is the purpose of the change
`TypeInformation` is a legacy class for the sole purpose of creating a
`TypeSerializer`.
Instances of `TypeInformation` are not required in the table ecosystem but
sometimes enforced
by interfaces of other modules (such as
`org.apache.flink.api.dag.Transformation`). Therefore, we recently introduced
`WrapperTypeInfo` which acts as an adapter whenever type information is
required. Instances of `WrapperTypeInfo` should only be created for passing it
to interfaces that require type information. The class should not be used as a
replacement for a `LogicalType`. Information such as the arity of a row type,
field types, field names, etc. should be derived from the `LogicalType`
directly.
This PR updates the code base to not use `RowDataTypeInfo` anymore.
`RowDataTypeInfo` was a composite type information with legacy types as field
types. This refactoring unblocks FLINK-15803 and reduces potential bugs from
type conversion.
## Brief change log
See commit messages.
## Verifying this change
This change is already covered by existing tests.
## Does this pull request potentially affect one of the following parts:
- Dependencies (does it add or upgrade a dependency): no
- The public API, i.e., is any changed class annotated with
`@Public(Evolving)`: yes
- The serializers: yes
- The runtime per-record code paths (performance sensitive): no
- Anything that affects deployment or recovery: JobManager (and its
components), Checkpointing, Kubernetes/Yarn/Mesos, ZooKeeper: no
- The S3 file system connector: no
## Documentation
- Does this pull request introduce a new feature? no
- If yes, how is the feature documented? not applicable
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
