Как правильно организовать Thread pool?

Question

[My1Name]

Есть метод public synchronized read(URL url). Он использует WebDriver selenium, а его объявление как "synchronized" почему-то не срабатывает... Чтоб обойти этот "баг", я пытаюсь создать пул очередности потоков следующим образом:

private static LinkedList<Runnable> pool;
..........................
public synchronized void read(URL url){
Runnable task = Thread.currentThread();
		pool.add(task);
		if (!pool.isEmpty() && pool.size()>1)
			try { task.wait();
			} catch (InterruptedException e1) {
				e1.printStackTrace();
			}
// body of method read
..........................
		  pool.removeFirst();
		  if (!pool.isEmpty()) {
			  Runnable first = pool.getFirst();
			  first.notify();
		  }
	    }

Таким образом получается

java.lang.IllegalMonitorStateException.

Как правильно организовать Thread pool?

Answer 1

[Elmo Sputterspark]

Правильно использовать пул из стандартной библиотеки, а не городить костыли. Как пулы связаны с синхронизацией вообще непонятно.

Comments

[My1Name]

Как пулы связаны с синхронизацией

- Метод использует WebDriver selenium, а его объявление как "synchronized" почему-то не срабатывает...

Драйвер объявлен статическим, и единственным экземпляром в классе, потому что создавать новый для каждого потока - это затратно (Жрёт ресурсы).

Что я уже только не делал: Создавал пул в виде карты на WindowHandle - ConcurrentHashMap; пытался организовать очередь через ExecutorService Future; совал потоки в BlockingQueue... В общем остановился на том, что при входе в метод у каждого свой new WebDriver Организовать очередь запущенных потоков - мне так и не удалось.

[Elmo Sputterspark]

My1Name, всё правильно, он же не является потокобезопасным, а потому не предназначен для эксплуатации в конкурентной среде.