Apache Allura™

Forge software for hosting software projects

Brought to you by: alexluberg, brondsem, ccruz, deshani, and 11 others

#6695 timeout & loop detection in LCD logic

Milestone: v1.0.1

Status: closed

Owner: Cory Johns

Labels: performance (126) sf-2 (994)

Component: General

Reviewer: Dave Brondsema

Updated: 2015-08-20

Created: 2013-09-20

Creator: Dave Brondsema

Private: 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 - 2013-09-20

status: open --> in-progress

assigned_to: Cory Johns
If you would like to refer to this comment somewhere else in this project, copy and paste the following link:

Cory Johns - 2013-09-20

status: in-progress --> code-review
If you would like to refer to this comment somewhere else in this project, copy and paste the following link:

Cory Johns - 2013-09-20

allura:cj/6695

If you would like to refer to this comment somewhere else in this project, copy and paste the following link:

Dave Brondsema - 2013-09-23

QA: Dave Brondsema
If you would like to refer to this comment somewhere else in this project, copy and paste the following link:

Dave Brondsema - 2013-09-23

status: code-review --> closed
If you would like to refer to this comment somewhere else in this project, copy and paste the following link:

Dave Brondsema - 2013-09-23

status: closed --> open
If you would like to refer to this comment somewhere else in this project, copy and paste the following link:

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.

If you would like to refer to this comment somewhere else in this project, copy and paste the following link:

Cory Johns - 2013-09-24

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

If you would like to refer to this comment somewhere else in this project, copy and paste the following link:

Cory Johns - 2013-09-24

status: open --> code-review
If you would like to refer to this comment somewhere else in this project, copy and paste the following link:

Dave Brondsema - 2013-09-24

status: code-review --> closed
If you would like to refer to this comment somewhere else in this project, copy and paste the following link:

Log in to post a comment.