Skip to content

Fix a bug for PollOnlyIfExecutorHasCapacity flag about pool size #666

Merged
longquanzheng merged 2 commits intocadence-workflow:masterfrom
cruxinformatics:switch-poller-to-max-poolsize
Jan 14, 2022
Merged

Fix a bug for PollOnlyIfExecutorHasCapacity flag about pool size #666
longquanzheng merged 2 commits intocadence-workflow:masterfrom
cruxinformatics:switch-poller-to-max-poolsize

Conversation

@spmistry
Copy link
Contributor

No description provided.

@coveralls
Copy link

Pull Request Test Coverage Report for Build 1061

  • 0 of 1 (0.0%) changed or added relevant line in 1 file are covered.
  • 2 unchanged lines in 2 files lost coverage.
  • Overall coverage increased (+0.04%) to 71.211%

Changes Missing Coverage Covered Lines Changed/Added Lines %
src/main/java/com/uber/cadence/internal/worker/PollTaskExecutor.java 0 1 0.0%
Files with Coverage Reduction New Missed Lines %
src/main/java/com/uber/cadence/internal/sync/WorkflowThreadContext.java 1 82.46%
src/main/java/com/uber/cadence/internal/worker/BlockCallerPolicy.java 1 62.5%
Totals Coverage Status
Change from base Build 1057: 0.04%
Covered Lines: 10839
Relevant Lines: 15221

💛 - Coveralls

@longquanzheng longquanzheng changed the title fix a bug with pool size value for capacity based polling Fix a bug for PollOnlyIfExecutorHasCapacity flag about pool size Jan 14, 2022
@longquanzheng longquanzheng merged commit 31524fa into cadence-workflow:master Jan 14, 2022
@spmistry spmistry deleted the switch-poller-to-max-poolsize branch January 20, 2022 02:12
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.

3 participants