Skip to content

Conversation

@Prathap-P
Copy link
Contributor

What this PR does / why we need it:

Which issue(s) this PR fixes:

Misc

@Prathap-P Prathap-P requested a review from a team as a code owner September 16, 2025 07:49
@franciscojavierarceo
Copy link
Member

@Prathap-P can you fix the DCO? Just have to click the UI.

@franciscojavierarceo franciscojavierarceo changed the title Add support for extra options for stream source feat: Add support for extra options for stream source Oct 26, 2025
@Prathap-P
Copy link
Contributor Author

What this PR does / why we need it:

This pr exposes runtime options that users can give before ingesting stream data.

Which issue(s) this PR fixes:

  • Our team have certificate way of authn/authz set for kafka topic and currently feast doesn't accept those options while registering a kafka source.
  • For ex: "kafka.security.protocol", "kafka.ssl.truststore.location", "kafka.ssl.truststore.password", "kafka.ssl.keystore.location", "kafka.ssl.keystore.password", "kafka.ssl.key.password"
  • So I thought we can expose an optional stream source options that users can provide while ingesting, so that they dont have to register again multiple times when options change.

Misc

I think this addition will be essential as spark supports numerous options which are used in real world scenarios.

@Prathap-P Prathap-P force-pushed the stream_source_options branch from a27fd0d to 77010bf Compare November 18, 2025 15:39
@Prathap-P
Copy link
Contributor Author

Prathap-P commented Nov 18, 2025

Hi @franciscojavierarceo DCO check is passed.
Let me know if you need anything from my side.

Copy link
Member

@franciscojavierarceo franciscojavierarceo left a comment

Choose a reason for hiding this comment

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

Lgtm

@ntkathole ntkathole merged commit 18956c2 into feast-dev:master Dec 2, 2025
17 checks passed
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.

3 participants