8

I have a situation that seems to fit the Async Servlet 3.0 / Comet situation but all I need to do is return a 200 response code (or other) after accepting the incoming parameters.

Is there a way for a HttpServlet to complete the http request/response handshake and yet continue processing?

Something like...

doPost( req, response ) {
   // verify input params...
   response.setStatus( SC_OK );
   response.close();
   // execute long query
}     

EDIT: Looking at the javax.servlet package - the proper phrasing to my question is

How do I commit a response?

as in Servlet.isCommitted()

Stevko
  • 4,345
  • 6
  • 39
  • 66
  • "Committed" means some part of the response stream has already been sent to the client. It can't be taken back or modified. It can happen as soon as you set a header or write to the response, in theory, but usually the container buffers so it happens a bit later. This is not relevant to you, I think. Don't close the response streams. Just set status, launch a thread as per other answers, let doPost() finish. – Sean Owen Mar 05 '10 at 12:15
  • I tried to use response.sendError( SC_OK ) but it still did not flush the response to the requestor. It appears that the only way to close a servlet request is to return from a doPost(). – Stevko Mar 05 '10 at 17:49

4 Answers4

6

Here's how I've handled this situation:

  1. When the app starts up, create an ExecutorService with Executors.newFixedThreadPool(numThreads) (there are other types of executors, but I suggest starting with this one)
  2. In doPost(), create an instance of Runnable which will perform the desired processing - your task - and submit it to the ExecutorService like so: executor.execute(task)
  3. Finally, you should return the HTTP Status 202 Accepted, and, if possible, a Location header indicating where a client will be able to check up on the status of the processing.

I highly recommend you read Java Concurrency in Practice, it's a fantastic and very practical book.

Avi Flax
  • 50,872
  • 9
  • 47
  • 64
  • While I agree with all the responses, this one has the least amount of complexity involved. Thanks to pajton, beny23 and avi for your input. – Stevko Mar 05 '10 at 17:48
  • how do I do this, if I want to use executor.submit() instead of .execute? I want to use .submit so that I can get hold of "futures" and know when all the tasks have completed (based on which I have to do certain processing?). – Tintin Jan 27 '17 at 16:10
3

You can continue processing in a separate Thread.

The response is commited once you return from doPost() method.

mac
  • 42,153
  • 26
  • 121
  • 131
pajton
  • 15,828
  • 8
  • 54
  • 65
  • I'm running into the URL Fetch service 5 sec timeout on perfectly good requests originating from a Google app engine. There is no need for the App Engine Requestor to block waiting for OK response from something that may take a very long time to complete processing. – Stevko Mar 02 '10 at 23:35
3

On possibility for your servlet to accept a request for processing in the background, is for the servlet to hand off processing to a separate thread which then executes in the background.

Using Spring, you can invoke a separate Thread using the a TaskExecutor. The advantage of using spring over standard JDK 5 java.util.concurrent.Executor is that if you're on application servers that need to use managed threads (IBM websphere or Oracle weblogic), you can use the WorkManagerTaskExecutor to hook into the CommonJ work managers.

Another alternative would be to move the long query logic into a Message Driven Bean or Message Driven POJO (Spring JMS can help here) and let the servlet simply post a message on a JMS queue. That would have the advantage that should the load on your web container become too great because of your long running query, you could easily move the MDB onto a different (dedicated) system.

beny23
  • 34,390
  • 5
  • 82
  • 85
0

This example can help

void doPost(){
    // do something
   final ExecutorService executor = Executors.newSingleThreadExecutor();
      executor.execute(new Runnable() {
          @Override
          public void run() {
              // processing after response
          }
      });}
wiem
  • 1