Project

General

Profile

Actions

Bug #14462

closed

when javascript engine timeout, we shutdown the thread before having a chance to report about it

Added by Nicolas CHARLES about 5 years ago. Updated almost 5 years ago.

Status:
Rejected
Priority:
N/A
Assignee:
-
Category:
Performance and scalability
Target version:
Severity:
UX impact:
User visibility:
Effort required:
Priority:
0
Name check:
Fix check:
Regression:

Description

in JavascriptEngine, we have a timeout of 5 secondes, and when we timeout, we terminate the thread
it can cause
```Exception caught during policy update process: Task java.util.concurrent.FutureTask@4cc89444 rejected from java.util.concurrent.ThreadPoolExecutor@7a2cae84[Terminated, pool size = 0, active threads = 0, queued tasks = 0, completed tasks = 24]``` but we are unsure

we need to add logging BEFORE shutown


Related issues 1 (0 open1 closed)

Related to Rudder - Bug #14465: Generation fails with "Task FutureTask rejected from ThreadPoolExecutor" due to timeout in JS computationReleasedFrançois ARMANDActions
Actions #1

Updated by Vincent MEMBRÉ about 5 years ago

  • Target version changed from 4.3.11 to 4.3.12
Actions #2

Updated by François ARMAND about 5 years ago

I'm not sure it is that that cause the reported error. It seems that the JS engine is (at leat trying to) reporting on its errors. On the other hand, the reported error is the same as https://issues.rudder.io/issues/14465.

So perhaps the JS timeout because of some load, but it's the load which caused the exception. Still, it's useful to know that both problem are/were corelated, perhaps it is a way to reproduce the `Task java.util.concurrent.FutureTask rejected from java.util.concurrent.ThreadPoolExecutor` problem.

Actions #3

Updated by François ARMAND about 5 years ago

  • Related to Bug #14465: Generation fails with "Task FutureTask rejected from ThreadPoolExecutor" due to timeout in JS computation added
Actions #4

Updated by Benoît PECCATTE almost 5 years ago

  • Subject changed from when javascript engine timeout, we shutdown the threa before having a chance to report about it to when javascript engine timeout, we shutdown the thread before having a chance to report about it
Actions #5

Updated by Vincent MEMBRÉ almost 5 years ago

  • Target version changed from 4.3.12 to 4.3.13
Actions #6

Updated by Vincent MEMBRÉ almost 5 years ago

  • Target version changed from 4.3.13 to 4.3.14
Actions #7

Updated by Vincent MEMBRÉ almost 5 years ago

  • Target version changed from 4.3.14 to 587
Actions #8

Updated by Vincent MEMBRÉ almost 5 years ago

  • Target version changed from 587 to 4.3.14
Actions #9

Updated by Alexis Mousset almost 5 years ago

  • Status changed from New to Rejected

It was fixed in 5.0 in #14465

Actions

Also available in: Atom PDF