Skip to content

Timeout Exception #316

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.

Sign up for GitHub

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

Jump to bottom
Closed
alallema opened this issue Jan 17, 2022 · 3 comments · Fixed by #438
Closed

Timeout Exception #316

alallema opened this issue Jan 17, 2022 · 3 comments · Fixed by #438
Labels
good first issue Good for newcomers

Comments

@alallema
Copy link
Contributor

alallema commented Jan 17, 2022 •
edited
Loading

Currently, the Java SDK doesn't use the TimeoutException.

On most of the SDK there is three main Exception:

  • ApiError
  • CommunicationError
  • TimeoutError

It would be nice to implement a timeout catch everywhere where it would be necessary.

You can check as an example the Python SDK where the exception is implemented and used.
@alallema alallema mentioned this issue Jan 17, 2022
Merged
@alallema alallema added the good first issue Good for newcomers label Jan 19, 2022
@trangntt-016
Copy link

HI @alallema I'm wondering if this issue is still open?

@alallema
Copy link
Contributor Author

alallema commented Mar 14, 2022 •
edited
Loading

HI @trangntt-016,
Thank you for your interest, yes this issue is still open!
Don't hesitate if you have any questions

@trangntt-016 trangntt-016 mentioned this issue Mar 19, 2022
Closed
3 tasks
@alallema alallema mentioned this issue Sep 14, 2022
Closed
@alallema alallema linked a pull request Oct 13, 2022 that will close this issue
Rewriting of the error handler #438
Merged
2 tasks
@alallema
Copy link
Contributor Author

Closed #438

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
good first issue Good for newcomers
Projects
None yet
Milestone
No milestone
2 participants
@alallema @trangntt-016