Skip to content
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

Fix the char zero ending panic in native config for streaming #706

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

andrei-ionescu
Copy link

@andrei-ionescu andrei-ionescu commented Aug 7, 2024

Given the max.poll.interval.ms property, that is mandatory in the streaming case (ie: stream-consumer), when retrieving that value from the native config it will receive a zero char ending string which cannot pe parsed into u64 leading to panic.

Solution is to trim out the zero ending char from the string so that it can be parsed into u64.

@andrei-ionescu andrei-ionescu changed the title Fix the char zero ending in native config Fix the char zero ending panic in native config for streaming Aug 7, 2024
@andrei-ionescu andrei-ionescu force-pushed the ai.fix_char_zero_ending branch from 5a25da2 to 976d56d Compare December 10, 2024 16:04
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.

1 participant