1

I'm trying run some javascripts in java. I'm not sure if the scripts are correct and I'd like to kill the the invocation after a time period. This is how i run the scripts.

returnMethod = invocableEngine.invokeFunction("main", input);

My idea was to run something like a deamon thread, which starts second thread with a timeout. The second thread - ScriptEvaluator runs the invocable function, described above. ScriptEvaluator implements Runnable.

ScriptEvaluator se = new ScriptEvaluator(r);
ExecutorService executor = Executors.newSingleThreadExecutor();
executor.invokeAll(Arrays.asList(Executors.callable(se)), timeout, TimeUnit.MILLISECONDS);
executor.shutdownNow();

Well it doesn't work. After timeout the Thread still runs.

Another requirement is that only one SkriptEvaluator thread is running.

EDIT: I've found something very interesting in shutdownNow(). When it doesn't guarantees the stopping of the Threads, is there guaranteed way to do that?

There are no guarantees beyond best-effort attempts to stop processing actively executing tasks. For example, typical implementations will cancel via Thread.interrupt(), so any task that fails to respond to interrupts may never terminate.

Thanks in advance

aphex
  • 3,372
  • 2
  • 28
  • 56
  • There is no guaranteed way to stop a Thread. Java threads must stop on their own in order for your program to stay safe. – Jon Lin Aug 29 '11 at 16:15

2 Answers2

2

You can test how ScriptEvaluator() responds to Thread.interrupt(). If it's well written, then executor.shutdownNow() will work. If it's not, then there is no perfect solution.

However, assuming your javascript doesn't interact with your main code a that much, you can use Thread.stop(). If you decide to do this, then make sure you understand why not to use Thread.stop() in general.

Michał Šrajer
  • 30,364
  • 7
  • 62
  • 85
0

Executor.shutdownNow() does not always work properly, perhaps you'll need Thread.stop() as the hardcore solution.

umbr
  • 440
  • 2
  • 4