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.

Sign up for GitHub

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

Jump to bottom

Very verbose output if a post-update has a batch process #3372

Closed
pfrenssen opened this issue Feb 8, 2018 · 1 comment · Fixed by #4090
Closed

Very verbose output if a post-update has a batch process #3372

pfrenssen opened this issue Feb 8, 2018 · 1 comment · Fixed by #4090
Assignees
@pfrenssen
Labels
area-updatedb drush9 type-task

Comments

@pfrenssen
Copy link
Member

When executing a batch process in a post-update the output included unnecessary lines about starting and completing each individual batch:

 [notice] Update started: joinup_subscription_post_update_subscribe_to_community_content
 [notice] 1000 memberships updated.
 [ok] Update completed: joinup_subscription_post_update_subscribe_to_community_content
 [notice] Update started: joinup_subscription_post_update_subscribe_to_community_content
 [notice] 2000 memberships updated.
 [ok] Update completed: joinup_subscription_post_update_subscribe_to_community_content
 [notice] Update started: joinup_subscription_post_update_subscribe_to_community_content
 [notice] 3000 memberships updated.
 [ok] Update completed: joinup_subscription_post_update_subscribe_to_community_content

It would be easier to read if the redundant lines are removed:

 [notice] Update started: joinup_subscription_post_update_subscribe_to_community_content
 [notice] 1000 memberships updated.
 [notice] 2000 memberships updated.
 [notice] 3000 memberships updated.
 [ok] Update completed: joinup_subscription_post_update_subscribe_to_community_content
@pfrenssen pfrenssen self-assigned this Feb 8, 2018
This was referenced Jun 1, 2019
Closed
Merged
@pfrenssen
Copy link
Member Author

Awesome, thanks for working on this 👍

weitzman pushed a commit that referenced this issue Jun 3, 2019
@claudiu-cristea @weitzman
Reduce the useless verbosity of batch update output (#3372) (#4090)
24dbb49 
* Provide a regression test (#3372)

* Reduce useless verbosity of output (#3372)

* Fix tests for Windows platforms.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@pfrenssen pfrenssen

Labels
area-updatedb drush9 type-task
Projects
None yet
Milestone
No milestone
1 participant
@pfrenssen