Closed akshaynanavare closed 6 months ago
I think you missed https://github.com/panjf2000/ants/blob/dev/multipool_func.go#L56-L58
I think you missed https://github.com/panjf2000/ants/blob/dev/multipool_func.go#L56-L58
@panjf2000 Yes, Added now. Thanks UT is already there for both of the funcs If you want explicitly to cover this code, I will add
All modified and coverable lines are covered by tests :white_check_mark:
Project coverage is 91.30%. Comparing base (
5c07bca
) to head (7b5ec33
). Report is 3 commits behind head on dev.
:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.
1. Are you opening this pull request for bug-fixs, optimizations or new feature?
Yes, for small optimization of multi pools.
2. Please describe how these code changes achieve your intention.
If lbs is invalid then we should immediately return from
NewMultiPool
func instead of creating pool arrays and then check for lbs validation3. Please link to the relevant issues (if any).
NA
4. Which documentation changes (if any) need to be made/updated because of this PR?
NA
4. Checklist