Skip to content
This repository has been archived by the owner on Dec 13, 2023. It is now read-only.

batch polling in client #3304

Merged
merged 2 commits into from Oct 31, 2022
Merged

batch polling in client #3304

merged 2 commits into from Oct 31, 2022

Conversation

jxu-nflx
Copy link
Contributor

Pull Request type

  • Bugfix
  • Feature
  • Refactoring (no functional changes, no api changes)
  • Build related changes (Please run ./gradlew generateLock saveLock to refresh dependencies)
  • WHOSUSING.md
  • Other (please describe):

NOTE: Please remember to run ./gradlew spotlessApply to fix any format violations.

Changes in this PR

Describe the new behavior from this PR, and why it's needed
Issue #
Support to batch polling tasks in client

Alternatives considered

Describe alternative implementation you have considered

* @param numSlots the number of permits to acquire
* @return {@code true} - if permit is acquired {@code false} - if permit could not be acquired
*/
public boolean acquireSlots(int numSlots) {
Copy link
Collaborator

Choose a reason for hiding this comment

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

Is it possible to delete the canPoll() method above now that this method is being used instead?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

ya, removed now

@jxu-nflx jxu-nflx merged commit 03f8d96 into main Oct 31, 2022
@jxu-nflx jxu-nflx deleted the client_batch_polling branch October 31, 2022 21:40
@sks1995
Copy link
Contributor

sks1995 commented Nov 2, 2022

@jxu-nflx When will the next release happen ?
We want to leverage batch polling support.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants