[FLINK-14438] Fix RowTypeInfo.equals()

• Type: Improvement
• Status: Open
• Priority: Not a Priority
• Resolution: Unresolved
• Component/s: API / Type Serialization System
• Labels: auto-deprioritized-major auto-deprioritized-minor pull-request-available
• Assignee: Unassigned
• Reporter: Timo Walther
• Created: 17/Oct/19 12:30
• Updated: 27/Nov/21 19:58

Description

As discussed in FLINK-12848, the equals method of RowTypeInfo causes confusion because it does not consider field names which makes it difficult to use the type in data structures (such as hash maps) or testing.

So far RowTypeInfo is marked as @PublicEvolving. However, it is not feasible to perform changes to RowTypeInfo.equals() in the current code base. The number of affected tests (>80) shows that this change is not trivial and would affect the stability of the legacy planner. However, most of the failing tests are in the legacy planner. It seems the Blink code has no problems with this change which is great! We suggest to apply the changes once we drop the legacy planner.

Issue Links

• is a child of: FLINK-14437 Drop the legacy planner
• links to: GitHub Pull Request #17935