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

Feature request: Support comby timeouts #31

Open
charles-gray opened this issue Aug 31, 2022 · 0 comments
Open

Feature request: Support comby timeouts #31

charles-gray opened this issue Aug 31, 2022 · 0 comments

Comments

@charles-gray
Copy link

For large numbers of comby operations on large source bases, it can take upwards of an hour to process all the files in my testing. This comes down dramatically (5mins!) if I parallelise my calls to comby with multiprocessing up to 32 ways. Unfortunately comby has a default 3 second timeout, and complicated files / very busy periods means that some files trip the timeout and don't get processed.

I've added the ability to set the comby timeout in my local branch and it now gives me correct results with the performance improvements.

Attaching the diff I used here as part of the feature request in case it's something interesting for the main project.
comby-timestamps.diff.txt

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

No branches or pull requests

1 participant