Java multithreading not closing threads

Question

I have the following code in a command line application. Once the loop completes, my app is still running. Why is it not shutting down. From the logs, I can see that endIndex has reached. But the app is still running?

Executor exec = Executors.newFixedThreadPool(4);
for (int i = startIndex; i <= endIndex; i++) {
        final String spURL = urlPart + i;

        Runnable requestHandler = new Runnable() {
            @Override
            public void run() {
                try {
                    getImageForURL(spURL, 0);
                } catch (IOException ex) {
                } catch (Exception ex) {
                }
            }
        };
        exec.execute(requestHandler);
    }

Can you make sure that the function getImageForURL returns every time ? — Santosh
– Santosh, Commented Nov 17, 2011 at 8:13

denis.solonenko · Accepted Answer · 2011-11-17 05:14:39Z

3

Try this

ExecutorService e = Executors.newFixedThreadPool(4);
...
e.shutdown();

answered Nov 17, 2011 at 5:14

denis.solonenko

11.8k2 gold badges32 silver badges23 bronze badges

Sign up to request clarification or add additional context in comments.

Comments

srikanta · Accepted Answer · 2011-11-17 05:16:33Z

0

You need to shutdown the executors after the loop (ideally in a finally block):
exec.shutdown() or exec.shutdownNow()

answered Nov 17, 2011 at 5:16

srikanta

3,0094 gold badges23 silver badges36 bronze badges

Collectives™ on Stack Overflow

Java multithreading not closing threads

2 Answers 2

Comments

Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

2 Answers 2

Comments

Comments

Your Answer

Sign up or log in

Post as a guest

Related