Skip to content

Enhancements of require> #1815

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
Open
wants to merge 3 commits into
base: master
Choose a base branch
from
Open

Enhancements of require> #1815

wants to merge 3 commits into from

Conversation

@yoyama
Copy link
Contributor

@yoyama yoyama commented Sep 15, 2023

This PR is port from Treasure Data on require>

  • Fix the failure related to resource limitation
  • Fix the failure related to multiple attempts for a session
  • Support delay kick in resource limited situation

@yoyama
WM-1654 enhance require> to avoid unexpected failure with max running…
972900e 
… attempt (#51)

* WM-1654 Fix require> with max running attempts
* Add test for require> with max attempt
@yoyama
WM-1655 support delay kick of require> with resource limit. (#57)
5c4496e 
WM-1655 support delay kick of `require>` with resource limit.
@yoyama
WM-1260 Fix require> with multiple attempts (#65)
2ab7b7b 
Fix WM-1260 require> issue with multiple attempts
@yoyama yoyama requested a review from szyn September 15, 2023 07:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@szyn szyn Awaiting requested review from szyn

At least 1 approving review is required to merge this pull request.

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

@yoyama