aurora-reviews mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Stephan Erb <>
Subject Re: Review Request 60306: Ensure Thermos is not overloaded by an unlimited number of lost processes
Date Wed, 21 Jun 2017 21:37:17 GMT

This is an automatically generated e-mail. To reply, visit:

(Updated June 21, 2017, 11:37 p.m.)

Review request for Aurora.


Add screenshots to illustrate the observed issue.

Repository: aurora


Included changes:

* Thermos may consider launched processes to be LOST. Instead of
  restarting those immediately, the restarts will now be at least
  `min_duration` seconds apart. Restarts will also be capped at the
  TOTAL_RUN_LIMIT of 100 restarts. This ensures neither Thermos nor the
  observer will be overloaded by checkpoints. The handling of the LOST
  state is now consistent with the handling of both FAILED and FINISHED.
* Mark the success_transition and failure_transition as private. They
  are only used within `TaskPlanner` itself.
* Fix documented default of `min_duration` (i.e 5s rather than 15s).


  docs/reference/ 6a9a3ff988dd2102aa9d22e27f22487f18423894 
  src/main/python/apache/thermos/common/ da5120f8f0c2489927a03e9d78ccb4f9b6eb275f

  src/test/python/apache/thermos/common/ 132c1ec4977143b79df8d13804370e76a553c3b9



./pants test.pytest src/test/python::

File Attachments (updated)



Stephan Erb

  • Unnamed multipart/alternative (inline, None, 0 bytes)
View raw message