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

Queue runner for 'group received' and 'delete group' actions #655

Closed
bjendres opened this issue Oct 26, 2022 · 6 comments
Closed

Queue runner for 'group received' and 'delete group' actions #655

bjendres opened this issue Oct 26, 2022 · 6 comments
Assignees
Milestone

Comments

@bjendres
Copy link
Member

The submission group update and the group closing process already have a queue runner to avoid PHP timeouts when the groups are large.

However, the same is needed for deleting groups and for marking them as received. Both cases also need a number of API calls that grows with the group size, and is therefore equally prone to PHP timeouts.

@bjendres bjendres added this to the 1.8 milestone Oct 26, 2022
@bjendres bjendres modified the milestones: 1.8.x, 1.9.x Feb 22, 2023
@bjendres bjendres modified the milestones: 1.9.x, 1.10, Scheduled Features Jun 22, 2023
@pminf
Copy link

pminf commented Nov 14, 2023

We also have to deal with this problem because we regularly have to close groups of around 8000 entries. Unfortunately, the user does not receive any feedback if the request generates a timeout. You only notice it within the group and that the status of the individual contributions no longer changes.

@bjendres
Copy link
Member Author

As a workaround , you can close large groups via command line (i.e. no timeout) with
cv api3 SepaAlternativeBatching.received txgroup_id=XX

That might also work for already received, incomplete groups, but please test well, I'm not sure about this one.

@pminf
Copy link

pminf commented Nov 14, 2023

OK, good to know. But only admins have access to the cli, so it's not a practicable workaround for us.

@bjendres
Copy link
Member Author

OK, good to know. But only admins have access to the cli, so it's not a practicable workaround for us.

I hope I can get to implementing this soon.

bjendres added a commit that referenced this issue Mar 6, 2024
@bjendres
Copy link
Member Author

bjendres commented Mar 6, 2024

@pminf Could you test if this version (c73e713) works for you? If you have large SEPA group support enabled, the "mark received" should now also be executed by a runner.

@pminf
Copy link

pminf commented Mar 22, 2024

@bjendres Works like charm: I've closed a group with over 7000 entries in one go without failure. Thanks!

bjendres added a commit that referenced this issue Mar 22, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants