Skip to content

chore: improve visz on 504: Error in partition - {"error": "Connection error while initiating partition request from unstructured_client", "message": "", "type": "partition_connection_error"}#338

Merged
yuming-long merged 3 commits intomainfrom
yuming/more_retry_011y_on_504
Apr 16, 2026
Merged

chore: improve visz on 504: Error in partition - {"error": "Connection error while initiating partition request from unstructured_client", "message": "", "type": "partition_connection_error"}#338
yuming-long merged 3 commits intomainfrom
yuming/more_retry_011y_on_504

Conversation

@yuming-long
Copy link
Copy Markdown
Contributor

@yuming-long yuming-long commented Apr 16, 2026

504: Error in partition - {"error": "Connection error while initiating
partition request from unstructured_client", "message": "", "type": "partition_connection_error"}

change: "ReadError after 5 retries over 312.4s" instead of "message": ""


Note

Low Risk
Behavior only changes error formatting/raising when the backoff budget is exhausted; normal retry and success paths are unchanged.

Overview
Improves observability when retries exceed max_elapsed_time by rethrowing the original exception type with a message that includes the number of attempts and total elapsed time (sync and async backoff paths), while preserving the original exception via chaining.

Bumps the generated SDK version/user-agent from 0.43.2 to 0.43.3.

Reviewed by Cursor Bugbot for commit 400e8e5. Bugbot is set up for automated code reviews on this repo. Configure here.

Copy link
Copy Markdown

@cursor cursor bot left a comment

Choose a reason for hiding this comment

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

Cursor Bugbot has reviewed your changes and found 2 potential issues.

Fix All in Cursor

❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.

Reviewed by Cursor Bugbot for commit 08dc8ed. Configure here.

Comment thread src/unstructured_client/utils/retries.py Outdated
Comment thread src/unstructured_client/_version.py
@yuming-long yuming-long merged commit 5171475 into main Apr 16, 2026
18 checks passed
@yuming-long yuming-long deleted the yuming/more_retry_011y_on_504 branch April 16, 2026 19:34
@awalker4
Copy link
Copy Markdown
Collaborator

Oops sorry, didn't see this in time. The version bump also needs to go in gen.yaml. Then you can copy the last block in RELEASES.md and change the values, it doesn't matter what it says in there. We just need to touch RELEASES to trigger a build

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.

3 participants