Skip to content

Fix issues with tootctl's parallelization and progress reporting#12093

Merged
Gargron merged 1 commit intomasterfrom
fix-tootctl-parallelize
Oct 7, 2019
Merged

Fix issues with tootctl's parallelization and progress reporting#12093
Gargron merged 1 commit intomasterfrom
fix-tootctl-parallelize

Conversation

@Gargron
Copy link
Copy Markdown
Member

@Gargron Gargron commented Oct 7, 2019

  • Failures of ActiveRecord::Base.connection_pool.with_connection method were not rescued
  • Progress bar jumped to full at the end even if not all items were processed
  • Deprecation warnings when incrementing progress bar past initial total

@Gargron Gargron added the bug Something isn't working label Oct 7, 2019
@Gargron Gargron merged commit 38b6c34 into master Oct 7, 2019
@Gargron Gargron deleted the fix-tootctl-parallelize branch October 7, 2019 02:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@ykzts ykzts ykzts approved these changes

Assignees

No one assigned

Labels

bug Something isn't working

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@Gargron @ykzts
⚡