Skip to content

increase timeout to 60 minutes #117

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

Merged
merged 1 commit into from
Sep 26, 2024
Merged

Conversation

jordan-homan
Copy link
Contributor

@jordan-homan jordan-homan commented Sep 26, 2024

A customer reported that large PDFs are still failing, even after the updated retry logic for the occasional 502/503s. The job client previously had an incorrect limit of 10 minutes for maximum execution time. This change increase the total client execution time to 60 minutes.

Testing

Ran custom integration tests with 2500 page PDFs running with the following settings:

const requestParams: PartitionParameters = {
            files: file,
            splitPdfPage: true,
            strategy: Strategy.HiRes,
            splitPdfAllowFailed: false,
            splitPdfConcurrencyLevel: 15
        };

        const res: PartitionResponse = await client.general.partition({
            partitionParameters: {
                ...requestParams
            },
        });

@jordan-homan jordan-homan marked this pull request as ready for review September 26, 2024 17:32
Copy link
Collaborator

@awalker4 awalker4 left a comment

Choose a reason for hiding this comment

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

Awesome!

@jordan-homan jordan-homan force-pushed the increase_timeout_to_60_min branch from 8a2f600 to 049c085 Compare September 26, 2024 18:30
@jordan-homan jordan-homan merged commit 51a54ff into main Sep 26, 2024
2 checks passed
@jordan-homan jordan-homan deleted the increase_timeout_to_60_min branch September 26, 2024 18:45
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.

2 participants