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
drush_backend_batch_process() should clear batch_get() #3773
Comments
I guess the code below is not sufficient? If we need more code tahts probably where it should go Lines 345 to 349 in eaf9c3e
|
subscribe |
Just wanted to include this as a potentially related issue https://www.drupal.org/node/3068196, piggybacked from https://www.drupal.org/project/search_api/issues/3019652#comment-13167514. It provides additional context around the discussion @pfrenssen posted in the description. |
Posted a PR implementing @pfrenssen's suggestion at #4886 |
For reference I discovered this while testing a fix for https://www.drupal.org/project/user_email_verification/issues/3245192. |
Fixed in #4886 |
Describe the bug
This is coming out of a bug report that was filed for Search API: Issue #3011340: Multiple index update causes drush config import to get stuck.
If a batch operation is added with
batch_set()
and then processed usingdrush_backend_batch_process()
then the list of batch operations is not cleared. If a second call todrush_backend_batch_process()
is made in the same request, then the original set of batch operations is requeued, leading to unexpected results.To Reproduce
Expected behavior
drush_backend_batch_process()
should clean upbatch_get()
after it is done.Workaround
System Configuration
The text was updated successfully, but these errors were encountered: