Skip to content

[Bug] Fix bug that task_worker_pool not work #4543

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
Merged
morningman merged 2 commits into from
Sep 8, 2020
Merged

[Bug] Fix bug that task_worker_pool not work #4543

morningman merged 2 commits into from
Sep 8, 2020

Conversation

@morningman
Copy link
Contributor

@morningman morningman commented Sep 7, 2020

Proposed changes

The number of thread initialized in task worker pool is not right.
This bug is introduced from #4440

Types of changes

  • Bugfix (non-breaking change which fixes an issue)

@morningman-cmy
[Bug] Fix bug that task_worker_pool not work
efb7c73 
The number of thread initialized in task worker pool is not right.
This bug is introduced from apache#4440
@morningman morningman added the kind/fix Categorizes issue or PR as related to a bug. label Sep 7, 2020
@morningman morningman self-assigned this Sep 7, 2020
EmmyMiao87
EmmyMiao87 approved these changes Sep 7, 2020
View reviewed changes
Copy link
Contributor

@EmmyMiao87 EmmyMiao87 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@EmmyMiao87 EmmyMiao87 added the approved Indicates a PR has been approved by one committer. label Sep 7, 2020
acelyc111
acelyc111 approved these changes Sep 7, 2020
View reviewed changes
Copy link
Member

@acelyc111 acelyc111 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@morningman morningman merged commit e55327b into apache:master Sep 8, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@acelyc111 acelyc111 acelyc111 approved these changes

@EmmyMiao87 EmmyMiao87 EmmyMiao87 approved these changes

Assignees

@morningman morningman

Labels

approved Indicates a PR has been approved by one committer. kind/fix Categorizes issue or PR as related to a bug.

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@morningman @acelyc111 @EmmyMiao87 @morningman-cmy