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

feat: enable users to define retry logic when using partition_via_api #2948

Open
Coniferish opened this issue Apr 29, 2024 · 0 comments
Open
Labels
enhancement New feature or request ingest

Comments

@Coniferish
Copy link
Collaborator

Coniferish commented Apr 29, 2024

Is your feature request related to a problem? Please describe.
partition_via_api does not accept arguments for defining the retry logic to be used by the python client. This means partition_via_api and unstructured-ingest (which depends on partition_via_api) have restricted functionality.

Describe the solution you'd like
In partition_via_api, sdk.general.partition() should be called with an optional RetryConfig: New parameters (strategy: str, backoff: BackoffStrategy, and retry_connection_errors: bool) will need to be added to partition_via_api since request_kwargs is used for all of the partition and chunking parameters. These new params should be used to init a RetryConfig object.

In addition to updating partition_via_api, unstructured-ingest also needs to be updated.

@Coniferish Coniferish added enhancement New feature or request ingest labels Apr 29, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request ingest
Projects
None yet
Development

No branches or pull requests

1 participant