Skip to content

Conversation

@rkhachatryan
Copy link
Contributor

@rkhachatryan rkhachatryan commented Dec 15, 2025

What is the purpose of the change

On timer, TimeIntervalJoin potentially performs a lot of work iterating through all it's records in state.

And when multiple timers are triggered by a single watermark, checkpoint can be delayed significantly, effectively increasing e2e latency.

Recently, splittable timers were introduced to Flink for a limited set of operators in FLINK-20217.

TimeIntervalJoin can benefit from it as well.

@flinkbot
Copy link
Collaborator

flinkbot commented Dec 15, 2025

CI report:

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

@pnowojski
Copy link
Contributor

I think change LGTM, but doesn't it lack test coverage? Are there some an existing tests for delayed watermarks?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants