Skip to content

[BP-2.3][FLINK-39966][table-planner] FlinkRelMdModifiedMonotonicity wrongly reports a non-time-attribute Top-1 Rank as insert-only#28515

Merged
lincoln-lil merged 2 commits into
apache:release-2.3from
lincoln-lil:cp23-FLINK-39720
Jun 24, 2026
Merged

[BP-2.3][FLINK-39966][table-planner] FlinkRelMdModifiedMonotonicity wrongly reports a non-time-attribute Top-1 Rank as insert-only#28515
lincoln-lil merged 2 commits into
apache:release-2.3from
lincoln-lil:cp23-FLINK-39720

Conversation

@lincoln-lil

Copy link
Copy Markdown
Contributor

This is a backport pr for release-2.3 branch.

@flinkbot

flinkbot commented Jun 23, 2026

Copy link
Copy Markdown
Collaborator

CI report:

Bot commands The @flinkbot bot supports the following commands:
  • @flinkbot run azure re-run the last Azure build

@spuru9

spuru9 commented Jun 23, 2026

Copy link
Copy Markdown
Contributor

Can you rename the title to signify it BP PR, like in #28471.
Also if its needed for other 2.x branches you may like to open them together as well.

@lincoln-lil lincoln-lil changed the title [FLINK-39966][table-planner] FlinkRelMdModifiedMonotonicity wrongly reports a non-time-attribute Top-1 Rank as insert-only [BP-2.3][FLINK-39966][table-planner] FlinkRelMdModifiedMonotonicity wrongly reports a non-time-attribute Top-1 Rank as insert-only Jun 23, 2026
@lincoln-lil

Copy link
Copy Markdown
Contributor Author

@flinkbot run azure

…rongly reports a non-time-attribute Top-1 Rank as insert-only

FLINK-34702 removed the dedicated StreamPhysicalDeduplicate metadata handler and
re-routed deduplication monotonicity through StreamPhysicalRank, but the new
dispatch guard only checked RankUtil.isDeduplication (Top-1 ROW_NUMBER without
rank number). It dropped the sortOnTimeAttributeOnly invariant the old node type
implicitly carried.

Tighten the guard to isDeduplication && sortOnTimeAttributeOnly restoring the
original logic.

This closes apache#28505.
@lincoln-lil

Copy link
Copy Markdown
Contributor Author

Amend the commit title to have a new hash to re-trigger the azure pipeline.

@lincoln-lil lincoln-lil merged commit ac21574 into apache:release-2.3 Jun 24, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants