#6695 timeout & loop detection in LCD logic

v1.0.1
closed
General
2015-08-20
2013-09-20
No

The last-commit-doc logic should have loop detection and a configurable timeout so that worst-case scenarios cannot tie up system resources

Related

Tickets: #6686
Tickets: #6707

Discussion

  • Cory Johns

    Cory Johns - 2013-09-20
    • status: open --> in-progress
    • assigned_to: Cory Johns
     
  • Cory Johns

    Cory Johns - 2013-09-20
    • status: in-progress --> code-review
     
  • Cory Johns

    Cory Johns - 2013-09-20

    allura:cj/6695

     
  • Dave Brondsema

    Dave Brondsema - 2013-09-23
    • QA: Dave Brondsema
     
  • Dave Brondsema

    Dave Brondsema - 2013-09-23
    • status: code-review --> closed
     
  • Dave Brondsema

    Dave Brondsema - 2013-09-23
    • status: closed --> open
     
  • Cory Johns

    Cory Johns - 2013-09-23

    I'm not sure this ticket should be re-opened. Unless we're sure there's a bug in this change, the issue remaining in production means the problem wasn't in the LCD logic after all. But I guess we can leave it open until we isolate the exact issue with verbose TMW logging.

     
  • Cory Johns

    Cory Johns - 2013-09-24

    How wrong I was. :-) Fix for the timeout force-pushed to allura:cj/6695

     
  • Cory Johns

    Cory Johns - 2013-09-24
    • status: open --> code-review
     
  • Dave Brondsema

    Dave Brondsema - 2013-09-24
    • status: code-review --> closed