Skip to content

Add support for AWS Kinesis Enhanced Fan-Out as an input#669

Closed
jbeemster wants to merge 10 commits intowarpstreamlabs:mainfrom
jbeemster:efo-support
Closed

Add support for AWS Kinesis Enhanced Fan-Out as an input#669
jbeemster wants to merge 10 commits intowarpstreamlabs:mainfrom
jbeemster:efo-support

Conversation

@jbeemster
Copy link
Contributor

This pull request introduces a new consumption mode for Kinesis with Enhanced FanOut support. What this lets us do is provide a dedicated consumption pipe for Bento instead of depending on shared polling.

As much as possible I have tried to re-use the existing Kinesis input architectures and have only added EFO in terms of consuming the events but checkpointing + batching have been left identical. Quite a lot of the logical patterns have also been copied from the polling implementation and then altered to support EFO.

Copy link
Collaborator

@jem-davies jem-davies left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Had a quick scan through - will look more in-depth very soon ... Also could we downgrade the log calls to debug maybe - it seems that is what the rest of the code does?

Comment on lines +388 to +390
if k.conf.EnhancedFanOut.ConsumerName != "" && k.conf.EnhancedFanOut.ConsumerARN != "" {
return nil, errors.New("cannot specify both consumer_name and consumer_arn in enhanced_fan_out config")
}
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Could you add a LintRule for this? It will catch misconfiguration earlier if so.

@jbeemster
Copy link
Contributor Author

@jem-davies we have re-opened this PR now from our official company fork (apologies) - #680

Its the same code with some minor tweaks that we have found during our testing. Its looking stable now however!

@jbeemster jbeemster closed this Jan 30, 2026
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.

2 participants

Comments