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

Provide some way of queuing batches #118

Closed
Daniel-Mietchen opened this issue Feb 1, 2020 · 7 comments
Closed

Provide some way of queuing batches #118

Daniel-Mietchen opened this issue Feb 1, 2020 · 7 comments

Comments

@Daniel-Mietchen
Copy link
Contributor

@Daniel-Mietchen Daniel-Mietchen commented Feb 1, 2020

Since only about two batches can run in parallel without causing errors, it would be useful to be able to queue batches, e.g. by sending them to QuickStatements even when in OAuth mode.

@Daniel-Mietchen

This comment has been minimized.

Copy link
Contributor Author

@Daniel-Mietchen Daniel-Mietchen commented Mar 15, 2020

Here is an overview of my recent batches to highlight why it would be useful to process batches via an orderly queue.
Screenshot_2020-03-15 Author Disambiguator copy

@arthurpsmith

This comment has been minimized.

Copy link
Owner

@arthurpsmith arthurpsmith commented Mar 30, 2020

I agree this is needed; I've been thinking about how to do it. I think it might be best to limit to just one active batch at a time (per user).

@arthurpsmith

This comment has been minimized.

Copy link
Owner

@arthurpsmith arthurpsmith commented Mar 31, 2020

Batches should now be queued - let me know if you run into trouble with this feature!

@Daniel-Mietchen

This comment has been minimized.

Copy link
Contributor Author

@Daniel-Mietchen Daniel-Mietchen commented Apr 1, 2020

Seems to work smoothly for the most part, except that "restart batch" functionality is lost, as per #130 .

@Daniel-Mietchen

This comment has been minimized.

Copy link
Contributor Author

@Daniel-Mietchen Daniel-Mietchen commented Apr 1, 2020

Another thing is that it would be nice to have "Remove from queue?" (or "Delete batch") functionality also for inactive but unfinished batches.

@arthurpsmith

This comment has been minimized.

Copy link
Owner

@arthurpsmith arthurpsmith commented Apr 1, 2020

Yup, I had a bit of tidying up to do. Remove from queue is there now (as is delete if you use the checkboxes on the batches_oauth page). And restart works (puts back in the queue or starts it if there's no batch running and it would be the oldest queued item.

@Daniel-Mietchen

This comment has been minimized.

Copy link
Contributor Author

@Daniel-Mietchen Daniel-Mietchen commented Apr 2, 2020

Seems all sorted out now — nice!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked pull requests

Successfully merging a pull request may close this issue.

None yet
2 participants
You can’t perform that action at this time.