You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@iggy.apache.org by "justinmclean (via GitHub)" <gi...@apache.org> on 2026/09/22 06:12:58 UTC
[I] Add the fail-fast connection option to the Rust and Python quick-starts [iggy]
justinmclean opened a new issue, #4261:
URL: https://github.com/apache/iggy/issues/4261
### Description
connect() blocks forever with no output when the server isn't reachable. The default reconnection policy is unlimited retries, and the only trace of a retry is a tracing::info! call that's silent without a subscriber, so a newcomer running the quick-start against a server they forgot to start sees a program that never returns and no reason why. A fail-fast option already exists (reconnection_retries=0 on the connection string, or .with_reconnection_max_retries(Some(0)) on the Rust builder); it's just not in the quick-start docs. The fix is to add it to the quick-start snippets (sdk/rust/intro.mdx and sdk/python/intro.mdx) so a copy-pasted example gets a bounded connect and a clear error instead of a silent hang. This is a good first issue: a small, well-defined docs change with the exact files and the existing option named.
### Affected area / component
Documentation
### Proposed solution
Add the existing fail-fast option to the quick-start snippets (`sdk/rust/intro.mdx`, `sdk/python/intro.mdx`) so a newcomer copy-pasting the example gets a bounded connect and a clear error instead of a silent hang.
### Alternatives considered
Changing the SDK default from unlimited retries to a bounded value. Unlimited retries is right for a long-running service, and the Python docstring already flags this default as wrong for request/reply usage, so a docs fix beats a breaking default change.
### Contribution
- [ ] I'm willing to submit a pull request to implement this feature
### Good first issue
- [x] I think this could be a good first issue for a new contributor
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: commits-unsubscribe@iggy.apache.org.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [I] Add the fail-fast connection option to the Rust and Python quick-starts [iggy]
Posted by "justinmclean (via GitHub)" <gi...@apache.org>.
justinmclean commented on issue #4261:
URL: https://github.com/apache/iggy/issues/4261#issuecomment-6427945265
@shivaansh0610-LUFFY Sorry for the delay. Assigned to you. Are you still planning to work on this? The quick-starts are in apache/iggy-website.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: commits-unsubscribe@iggy.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [I] Add the fail-fast connection option to the Rust and Python quick-starts [iggy]
Posted by "slbotbm (via GitHub)" <gi...@apache.org>.
slbotbm commented on issue #4261:
URL: https://github.com/apache/iggy/issues/4261#issuecomment-6451054922
@QIW9 assigned
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: commits-unsubscribe@iggy.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [I] Add the fail-fast connection option to the Rust and Python quick-starts [iggy]
Posted by "slbotbm (via GitHub)" <gi...@apache.org>.
slbotbm commented on issue #4261:
URL: https://github.com/apache/iggy/issues/4261#issuecomment-6437662269
I unassigned the person since they abandoned a previous PR that they had opened.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: commits-unsubscribe@iggy.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [I] Add the fail-fast connection option to the Rust and Python quick-starts [iggy]
Posted by "jiengup (via GitHub)" <gi...@apache.org>.
jiengup commented on issue #4261:
URL: https://github.com/apache/iggy/issues/4261#issuecomment-5827246129
@shivaansh0610-LUFFY
Sure, you can go ahead with it.
I'll focus on #4262.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: commits-unsubscribe@iggy.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [I] Add the fail-fast connection option to the Rust and Python quick-starts [iggy]
Posted by "shivaansh0610-LUFFY (via GitHub)" <gi...@apache.org>.
shivaansh0610-LUFFY commented on issue #4261:
URL: https://github.com/apache/iggy/issues/4261#issuecomment-5826951865
Hyy @justinmclean ,
I think this is separate from #4262. That one's about the login handshake having no read timeout (a code fix) — this is just adding the existing reconnection_retries=0 / with_reconnection_max_retries(Some(0)) option to the two quick-start snippets. Confirmed the default is unlimited retries and the retry log is silent without a subscriber, so the doc gap is real.
@jiengup does your PR for #4262 also plan to touch the quick-start docs, or is this still open?
Couldn't find sdk/rust/intro.mdx in apache/iggy itself — is the docs site a separate repo?
Can you point me to it and assign this to me?
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: commits-unsubscribe@iggy.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [I] Add the fail-fast connection option to the Rust and Python quick-starts [iggy]
Posted by "jiengup (via GitHub)" <gi...@apache.org>.
jiengup commented on issue #4261:
URL: https://github.com/apache/iggy/issues/4261#issuecomment-5776170872
This will be covered as part of #4262.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: commits-unsubscribe@iggy.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [I] Add the fail-fast connection option to the Rust and Python quick-starts [iggy]
Posted by "QIW9 (via GitHub)" <gi...@apache.org>.
QIW9 commented on issue #4261:
URL: https://github.com/apache/iggy/issues/4261#issuecomment-6450412472
@slbotbm ,can I have a try? please assgin to me.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: commits-unsubscribe@iggy.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org