[FLINK-15001] The digest of sub-plan reuse should contain retraction traits for stream physical nodes

• Type: Bug
• Status: Closed
• Priority: Major
• Resolution: Fixed
• Affects Version/s: 1.9.0, 1.9.1
• Fix Version/s: 1.9.2, 1.10.0
• Component/s: Table SQL / Planner
• Labels: pull-request-available
• Assignee: godfrey he
• Reporter: godfrey he
• Created: 02/Dec/19 02:55
• Updated: 16/Dec/19 12:02
• Resolved: 14/Dec/19 03:28

Description

This bug is found in FLINK-14946:

The plan for the given sql in FLINK-14946 is

however, the plan after sub-plan reuse is:

in the first picture, we could find that the accMode of two joins are different, but the two joins are reused in the second picture.

The reason is the digest of sub-plan reuse does not contain retraction traits for stream physical nodes now.

Issue Links

• relates to: FLINK-14946 Retraction infer would result in bad plan under corner case in blink planner
• links to: GitHub Pull Request #10377
• links to: GitHub Pull Request #10561