Skip to content

[FLINK-37908] Resolve EFO-enabled KinesisStreamsSource initialization failure on automated restarts #213

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

digitalpoetry
Copy link

@digitalpoetry digitalpoetry commented Jun 6, 2025

Purpose of the change

Resolve race condition on automated restarts of Flink apps with an EFO-enabled KinesisStreamsSource.

Verifying this change

This change is a trivial rework / code cleanup without any test coverage.

This change is aligned with a previously implemented fix for the same issue observed within the FlinkKinesisConsumer connector:

Significant changes

(Please check any boxes [x] if the answer is "yes". You can first publish the PR and check them afterwards, for convenience.)

  • Dependencies have been added or upgraded
  • Public API has been changed (Public API is any class annotated with @Public(Evolving))
  • Serializers have been changed
  • New feature has been introduced
    • If yes, how is this documented? (not applicable / docs / JavaDocs / not documented)

Copy link

boring-cyborg bot commented Jun 6, 2025

Thanks for opening this pull request! Please check out our contributing guidelines. (https://flink.apache.org/contributing/how-to-contribute.html)

@digitalpoetry digitalpoetry changed the title FLINK-37908 Resolve EFO-enabled KinesisStreamsSource initialization failure on automated restarts [FLINK-37908] Resolve EFO-enabled KinesisStreamsSource initialization failure on automated restarts Jun 6, 2025
@davidradl
Copy link

@digitalpoetry you have format violations in CI checks .

@digitalpoetry
Copy link
Author

digitalpoetry commented Jun 17, 2025

For some reason, I was able to build on my work machine, but not my personal machine 🤷 I'll need to look into that...
Fixed now, thank you @davidradl

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.

2 participants