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.

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

Job Queue: Negative Timeout Value Causes Error in logs #30348

Open
jgambarios opened this issue Oct 15, 2024 · 2 comments · Fixed by #30350
Open

Job Queue: Negative Timeout Value Causes Error in logs #30348

jgambarios opened this issue Oct 15, 2024 · 2 comments · Fixed by #30350
Labels
Merged QA : Passed Internal Team : Scout Type : Task

Comments

@jgambarios
Copy link
Contributor

jgambarios commented Oct 15, 2024 •
edited
Loading

Task

The application encountered an error with the message java.lang.IllegalArgumentException: timeout value is negative. This error occurs because the code attempts to use a negative value for a timeout, which is not allowed.

The problem is due an overflow issue in the exponential backoff calculation, where the value became too large to handle, causing it to wrap around to a negative number.

Steps to Reproduce:

  1. Run the job queue manager for a long period of time .
  2. Observe the error in the logs.

Proposed Objective

Core Features

Proposed Priority

Priority 2 - Important
@jgambarios jgambarios self-assigned this Oct 15, 2024
jgambarios added a commit that referenced this issue Oct 15, 2024
@jgambarios
#30348 Add calculateBackoffTime method and unit tests
57addcc 
Introduced the calculateBackoffTime method to manage exponential backoff in job processing. Added corresponding unit tests and implemented a reset mechanism for the emptyQueueCount to improve reliability.
@github-actions GitHub Actions
Copy link

PRs:

@jgambarios jgambarios linked a pull request Oct 15, 2024 that will close this issue
Fix (Core) Fixed calculateBackoffTime in JobQueueManagerAPIImpl #30350
Merged
@github-actions github-actions bot mentioned this issue Oct 15, 2024
Merged
@jgambarios jgambarios reopened this Oct 15, 2024
@jgambarios jgambarios removed their assignment Oct 15, 2024
@fabrizzio-dotCMS fabrizzio-dotCMS self-assigned this Oct 21, 2024
@fabrizzio-dotCMS fabrizzio-dotCMS removed their assignment Oct 21, 2024
@fabrizzio-dotCMS
Copy link
Contributor

After having executed the LargeReadFile Processor for a long time I couldn't reproduce the issue.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
Merged QA : Passed Internal Team : Scout Type : Task
Projects
dotCMS - Product Planning
Status: QA - Backlog
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Fix (Core) Fixed calculateBackoffTime in JobQueueManagerAPIImpl dotCMS/core
3 participants
@jgambarios @nollymar @fabrizzio-dotCMS