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

Timeout Option for SchemaRegistryClient #905

Open
2 of 7 tasks
snazzyfox opened this issue Jun 30, 2020 · 4 comments · May be fixed by #1682
Open
2 of 7 tasks

Timeout Option for SchemaRegistryClient #905

snazzyfox opened this issue Jun 30, 2020 · 4 comments · May be fixed by #1682

Comments

@snazzyfox
Copy link

snazzyfox commented Jun 30, 2020

Description

SchemaRegistryClient does not supply a timeout to the underlying calls to requests. If a network issue causes no data to be returned (such as firewall/security group misconfiguration), the client will wait forever instead of raising a timeout error. This essentially causes the application to hang.

We should add a new configuration option to SchemaRegistryClient that sets the timeout so that an error may be raised/caught in these situations.

How to reproduce

With a blackhole server running locally, for example on port 8000

from confluent_kafka.schema_registry import SchemaRegistryClient
client = SchemaRegistryClient({'url': 'localhost:8000'})
client.get_subjects()  # this will hang forever

Checklist

Please provide the following information:

  • confluent-kafka-python and librdkafka version (confluent_kafka.version() and confluent_kafka.libversion()): both 1.4.2
  • Apache Kafka broker version: irrelevant
  • Client configuration: {...}: irrelevant
  • Operating system: Probably irrelevant since schema registry client is pure python, but reproduced on ubuntu 20.04, amazon linux 2, and windows 10
  • Provide client logs (with 'debug': '..' as necessary)
  • Provide broker log excerpts
  • Critical issue
@zschumacher
Copy link

Any update on this? It is still an issue for us at @simplebet and probably others

@jliunyu
Copy link
Contributor

jliunyu commented Feb 1, 2022

We are looking into this issue.

@slominskir
Copy link
Contributor

Duplicate of #622

@okhaliavka okhaliavka linked a pull request Nov 25, 2023 that will close this issue
@chipola
Copy link

chipola commented Mar 27, 2024

It looks like there's a PR submitted, can we get someone from the team to take a look? cc: @jliunyu

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

5 participants