Как обрабатывать исключение Jetty - длительный срок действия HTTP-запроса, но процесс, который он вызывает, никогда не заканчивается, а Jetty недовольна

У меня есть сервер Jetty, который обрабатывает длинные HTTP-запросы - ответы генерируются другим процессом X и заканчиваются хешем коллектора, который периодически запрашивает Jetty.

Есть 3 случая:

  • Процесс X заканчивается до истечения периода ожидания HTTP-запроса - нет проблем
  • Процесс X заканчивается после периода ожидания запроса - нет Проблема
  • Процесс X никогда не заканчивается - происходит исключение ниже

Как определить эту ситуацию (3) и предотвратить исключение, позволяя другим двум делам правильно работать?

Исключение:

2012-06-18 00:13:31.055:WARN:oejut.QueuedThreadPool:
java.lang.IllegalStateException: IDLE,initial
    at org.eclipse.jetty.server.AsyncContinuation.complete(AsyncContinuation.java:569)
    at server.AsyncHTTPRequestProcessor.run(AsyncHTTPRequestProcessor.java:72)
    at org.eclipse.jetty.server.handler.ContextHandler.handle(ContextHandler.java:1119)
    at org.eclipse.jetty.server.AsyncContinuation$1.run(AsyncContinuation.java:875)
    at org.eclipse.jetty.util.thread.QueuedThreadPool.runJob(QueuedThreadPool.java:599)
    at org.eclipse.jetty.util.thread.QueuedThreadPool$3.run(QueuedThreadPool.java:534)
    at java.lang.Thread.run(Thread.java:679)


Продолжение Jetty HTTP-запроса:

public class AsyncHTTPRequestProcessor implements Runnable {

    private ConcurrentHashMap<String, String> collector;
    private Logger logger;
    private AsyncContext ctx;
    //Defined this here because of strange behaviour when running junit
    //tests and the response json string being empty...
    private String responseStr = null;

    public AsyncHTTPRequestProcessor(AsyncContext _ctx, 
            ConcurrentHashMap<String, String> _collector, Logger _logger) {
        ctx = _ctx;
        collector = _collector;
        logger = _logger;
    }

    @Override
    public void run() {

        logger.info("AsyncContinuation start");

        //if(!((AsyncContinuation)ctx).isInitial()){
        String rid = (String) ctx.getRequest().getAttribute("rid");
        int elapsed = 0;
        if(rid !=null)
        {

            logger.info("AsyncContinuation rid="+rid);

            while(elapsed<ctx.getTimeout())
            {
                if(collector.containsKey(rid)){
                    responseStr = collector.get(rid);
                    collector.remove(rid);

                    logger.info("--->API http request in collector:"+responseStr);
                    ctx.getRequest().setAttribute("status",200);
                    ctx.getRequest().setAttribute("response", responseStr);
                    ctx.getRequest().setAttribute("endTime",System.currentTimeMillis());
                    //ctx.complete();
                    break;
                }
                try {
                    Thread.sleep(10);
                    elapsed+=10;
                } catch (InterruptedException e) {
                    e.printStackTrace();
                }
            }
            //}
            logger.info("Collector in async stuff:");
            for(String key:collector.keySet()){
                logger.info(key+"->"+collector.get(key));
            }

            for(Entry<String, String> x:collector.entrySet()){
                logger.info(x.getKey()+"->"+x.getValue());
            }
            ctx.complete(); <---- this line 72
        }
    }

}

ОТВЕТЫ

Ответ 1

Проблема здесь заключается не в вашем вызове AsyncContext # complete(), а в надстрочном дизайне кода.

Продолжение (то же самое для ассемблера сервлета) имеет асинхронный. Цикл while, который использует тайм-аут внутреннего продолжения, не должен быть здесь. При этом вы преобразуете асинхронный дизайн в синхронный. Правильная вещь - зарегистрировать слушателя с помощью функции продолжения # addContinuationListener() и реализует метод onTimeout() для надлежащего обработки тайм-аута.

Как только ваша логика тайм-аута исчезнет, ​​я бы рекомендовал переместить логику процесса X в класс AsyncHTTPRequestProcessor и отказаться от необходимости использования коллектора. Во время обработки вы должны предположить, что текущий поток никогда не будет отключен. Делая это, ваш вызов complete() делает чувство, и вы будете защищены от проблемы concurrency на коллекционере.

Ответ 2

Использование блока catch try может помочь в этом случае.

   try{
      ctx.complete()
   } catch (IllegalStateException e){
      //Handle it the way you prefer.
   }