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

correct the number of selected invoker #5875

Merged
merged 1 commit into from
Apr 1, 2020
Merged

correct the number of selected invoker #5875

merged 1 commit into from
Apr 1, 2020

Conversation

Augustvic
Copy link
Contributor

@Augustvic Augustvic commented Mar 16, 2020

What is the purpose of the change

To make sure the number of selected invoker is as specified in the parameter of "forks".

Brief changelog

In the previous version, there was no guarantee that enough invokers were selected because of the “if” statement. It can only ensure that the loop is executed “forks” times.

Verifying this change

XXXXX

@Augustvic Augustvic changed the title Update ForkingClusterInvoker.java correct the number of selected invoker Mar 16, 2020
@chickenlj chickenlj merged commit 61d5d44 into apache:master Apr 1, 2020
@chickenlj chickenlj added this to the 2.7.7 milestone Apr 1, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants