river-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Bryan Thompson <br...@systap.com>
Subject Re: test failure repeatability - TaskManager
Date Tue, 02 Apr 2013 21:17:53 GMT
I am not clear on the semantics for runAfter, but maybe this can be
achieved by wrapping a Runnable within another Runnable such that the 2nd
runnable is automatically scheduled after the first has succeeded?
Likewise, it is possible to wrap a Runnable in order to automatically
retry if it throws an exception.

There are people who are experts at these patterns, but an example is
given (below my signature) for an Executor that wraps an ExecutorService
and queues Runnable instances with limited parallelism.  It hooks the
Runnable in its own run() method.

If you use a ScheduledExecutorService, you can queue a task to run with an
initial and repeated delay (or at a repeated interval).  The task will be
rescheduled *unless* it throws an exception.  This could be reused to
periodically run-try a task after a timeout if we convert an error thrown
in the task into "no error" (hence run after a fixed delay) and throw out
a known exception if there is no error (to terminate the retry of the
task).  A bit of a hack, but it leverages existing code for re-running a
task with a fixed delay.


package com.bigdata.util.concurrent;

import java.util.concurrent.BlockingQueue;
import java.util.concurrent.Callable;
import java.util.concurrent.Executor;
import java.util.concurrent.ExecutorService;
import java.util.concurrent.Future;
import java.util.concurrent.FutureTask;
import java.util.concurrent.LinkedBlockingDeque;
import java.util.concurrent.RejectedExecutionException;
import java.util.concurrent.Semaphore;

import org.apache.log4j.Logger;

 * A fly weight helper class that runs tasks either sequentially or with
 * parallelism against some thread pool. Deadlock can arise when limited
 * parallelism is applied if there are dependencies among the tasks.
 * parallelism is enforced using a counting {@link Semaphore}. New tasks
 * start iff the latch is non-zero. The maximum parallelism is the minimum
 * the value specified to the constructor and the potential parallelism of
 * delegate service.
 * <p>
 * Note: The pattern for running tasks on this service is generally to
 * {@link #execute(Runnable)} a {@link Runnable} and to make that
 * {@link Runnable} a {@link FutureTask} if you want to await the {@link
 * of a {@link Runnable} or {@link Callable} or otherwise manage its
 * <p>
 * Note: This class can NOT be trivially wrapped as an {@link
 * since the resulting delegation pattern for submit() winds up invoking
 * execute() on the delegate {@link ExecutorService} rather than on this
 * @author <a href="mailto:thompsonbry@users.sourceforge.net">Bryan
 * @version $Id: LatchedExecutor.java 6749 2012-12-03 14:42:48Z
thompsonbry $
public class LatchedExecutor implements Executor {

    private static final transient Logger log = Logger
     * The delegate executor.
    private final Executor executor;
     * This is used to limit the concurrency with which tasks submitted to
     * class may execute on the delegate {@link #executor}.
    private final Semaphore semaphore;
     * A thread-safe blocking queue of pending tasks.
     * @todo The capacity of this queue does not of necessity need to be
     *       unbounded.
    private final BlockingQueue<Runnable> queue = new

    private final int nparallel;
     * Return the maximum parallelism allowed by this {@link Executor}.
    public int getNParallel() {
    	return nparallel;
    public LatchedExecutor(final Executor executor, final int nparallel) {

        if (executor == null)
            throw new IllegalArgumentException();

        if (nparallel < 1)
            throw new IllegalArgumentException();

        this.executor = executor;

        this.nparallel = nparallel;
        this.semaphore = new Semaphore(nparallel);


    public void execute(final Runnable r) {
        if (!queue.offer(new Runnable() {
             * Wrap the Runnable in a class that will start the next
             * from the queue when it completes.
            public void run() {
                try {
                } finally {
        })) {
            // The queue is full.
            throw new RejectedExecutionException();
        if (semaphore.tryAcquire()) {
            // We were able to obtain a permit, so start another task.

     * Schedule the next task if one is available (non-blocking).
     * <p>
     * Pre-condition: The caller has a permit.
    private void scheduleNext() {
        while (true) {
            Runnable next = null;
            if ((next = queue.poll()) != null) {
                try {
                } catch (RejectedExecutionException ex) {
                    // log error and poll the queue again.
                    log.error(ex, ex);
            } else {


View raw message