Allow setting task execution timeout as a commandline parameter

arkorsky / jodconverter

Automatically exported from code.google.com/p/jodconverter

0 stars 0 forks source link

Allow setting task execution timeout as a commandline parameter #32

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago

What steps will reproduce the problem?
* Run a task that takes more than 5 minutes

What is the expected output? What do you see instead?
* I would like to be able to run with arbitrary timeout length, without
having to recompile the .jar

What version of the product are you using? On what operating system?
* 3.0-beta-1 on linux (rhel5)

Please provide any additional information below.
* Great product.  I am a sysadmin, not a programmer, and so I do not feel
confident enough to submit a patch that would be usable (right now I can
just modify the constant and rebuild the .jar, but I think this should be a
commandline option).

Original issue reported on code.google.com by joefresh...@gmail.com on 12 Feb 2009 at 6:59

GoogleCodeExporter commented 9 years ago

Not sure we want to add a command line switch for every possible configuration 
option.

The command line tool should probably have no task execution timeout set by 
default
though.

Original comment by mirko.na...@gmail.com on 11 Apr 2009 at 2:32

GoogleCodeExporter commented 9 years ago

Issue 61 has been merged into this issue.

Original comment by mirko.na...@gmail.com on 15 Jan 2010 at 4:42

GoogleCodeExporter commented 9 years ago

Issue 67 has been merged into this issue.

Original comment by mirko.na...@gmail.com on 15 May 2010 at 11:05

GoogleCodeExporter commented 9 years ago

Original comment by mirko.na...@gmail.com on 15 May 2010 at 11:06

Changed state: Accepted
Added labels: Target-3.0-rc-1, Type-Enhancement, Version-3.0-beta-3

GoogleCodeExporter commented 9 years ago

From my understanding, you want to add the task execution timeout. The proposed 
patch in Issue 67 looks valid. Do you want me to include this in the sigar 
branch and close this issue? Or did you have some other implementation in mind?

Original comment by shervin.asgari@gmail.com on 23 Mar 2011 at 12:39

GoogleCodeExporter commented 9 years ago

Yep feel free to add to the Sigar branch, thanks.

Original comment by mirko.na...@gmail.com on 23 Mar 2011 at 12:45

GoogleCodeExporter commented 9 years ago

Added patch from Issue 67 to sigar branch

Original comment by shervin.asgari@gmail.com on 23 Mar 2011 at 12:50

Changed state: Fixed

GoogleCodeExporter commented 9 years ago

Original comment by mirko.na...@gmail.com on 27 Mar 2011 at 1:10

Changed state: FixedInBranch

GoogleCodeExporter commented 9 years ago

Original comment by mirko.na...@gmail.com on 27 Mar 2011 at 1:12

Added labels: Target-3.0-beta-4

GoogleCodeExporter commented 9 years ago

Original comment by mirko.na...@gmail.com on 7 May 2011 at 9:21

Changed state: Fixed

GoogleCodeExporter commented 9 years ago

Original comment by mirko.na...@gmail.com on 7 May 2011 at 3:32

Added labels: Priority-Low