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

feat(delay-operator): support long delays by segmenting the delay (#7440) #7441

Closed
wants to merge 1 commit into from

Conversation

danilipari
Copy link

Description:
This commit updates the delay operator to handle delays longer than 2147483647 ms (~24.9 days), addressing the JavaScript timer limitation. It introduces a segmented delay approach, allowing the delay operator to accurately manage long delay periods without exceeding the maximum limit. This change ensures reliable delay operations for scenarios requiring long wait times, such as token expiration or delayed notifications, improving the operator's versatility and reliability.

Related issue (7440): #7440

…ing the delay

This commit updates the delay operator to handle delays longer than 2147483647 ms (~24.9 days), addressing the JavaScript timer limitation. It introduces a segmented delay approach, allowing the delay operator to accurately manage long delay periods without exceeding the maximum setTimeout/setInterval limit. This change ensures reliable delay operations for scenarios requiring long wait times, such as token expiration or delayed notifications, improving the operator's versatility and reliability.
@danilipari danilipari changed the title feat(delay-operator): support long delays beyond 24.9 days by segmenting the delay (#7440) feat(delay-operator): support long delays by segmenting the delay (#7440) Feb 10, 2024
@kwonoj
Copy link
Member

kwonoj commented Feb 10, 2024

Refer #3015, core team's general agreement is we want to doc as known limitation.

@danilipari danilipari closed this Feb 10, 2024
@danilipari danilipari deleted the fix-issue-7440 branch February 10, 2024 15:57
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.

None yet

2 participants