Skip to content

[Backport][FLINK-38916][FLINK-38417][FLINK-38445][FLINK-39015][table] Backport MultiJoin fixes to 2.1#27908

Merged
snuyanzin merged 4 commits intoapache:release-2.1from
gustavodemorais:FLINK-38916-backport-2.1
Apr 14, 2026
Merged

[Backport][FLINK-38916][FLINK-38417][FLINK-38445][FLINK-39015][table] Backport MultiJoin fixes to 2.1#27908
snuyanzin merged 4 commits intoapache:release-2.1from
gustavodemorais:FLINK-38916-backport-2.1

Conversation

@gustavodemorais
Copy link
Copy Markdown
Contributor

  • FLINK-38916 - cherry-pick with XML conflict (regenerated)
  • FLINK-38417 - added as dependency. Test conflict in MultiJoinSemanticTests.java resolved by keeping existing test programs from 2.1 (e.g. MIXED_CHANGELOG_MODES) and adding the new ones from the cherry-pick (e.g. MULTI_KEY_TYPES)
  • FLINK-38445 - added as dependency (XML conflict, regenerated)
  • FLINK-39015 - clean cherry-pick after dependencies added
  • Fixed: isHeapBackend() adapted for 2.1 API (instanceof HeapKeyedStateBackend instead of getBackendTypeIdentifier()), removed duplicate test methods, replaced addMode() with addOption("changelog-mode", ...)

Minor conflicts mostly related to tests and one naming change.

@flinkbot
Copy link
Copy Markdown
Collaborator

flinkbot commented Apr 9, 2026

CI report:

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

@gustavodemorais gustavodemorais changed the title [FLINK-38916][FLINK-38417][FLINK-38445][FLINK-39015][table] Backport MultiJoin fixes to 2.1 [Backport][FLINK-38916][FLINK-38417][FLINK-38445][FLINK-39015][table] Backport MultiJoin fixes to 2.1 Apr 13, 2026
@snuyanzin
Copy link
Copy Markdown
Contributor

thanks for backporting @gustavodemorais

for the future: can you please put links to actual commits in master rather than jira
it is much easier to check it then

@snuyanzin snuyanzin merged commit ef1952c into apache:release-2.1 Apr 14, 2026
@gustavodemorais
Copy link
Copy Markdown
Contributor Author

Hey @snuyanzin, sure. Thanks for the note, I'll try to keep that in mind.

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.

5 participants