sbobek / DynamicAdvertising

0 stars 0 forks source link

DEIncomingConnectionHandler - thread pool #5

Closed danius0001 closed 6 years ago

danius0001 commented 6 years ago

Tutaj podobnie Executor nie jest zamykany.

I tip na przyszłość: masz tu listę obiektów Future i odbierasz ich wyniki w kolejności występowania na liście. Przez co czekasz na wynik z pierwszego a w tym czasie może jakiś inny się skończyć -> marnuje się w ten sposób czas ;) polecam użycie usługi ExecutorCompletionService, która to obsługuje.

HrabiaVulpes commented 6 years ago

nie chcę wprowadzać teraz dużych zmian, zwłaszcza że w algorytmie potrzeba mi jakoś powiązać które DSP odesłało odpowiedź z odpowiedzią (dlatego odbieram wyniki w kolejności na liście) a ExecutorCompletionService.take() po prostu zwróci mi pierwsze które odpowiedziało, więc musiałbym wprowadzać jakąś specjalną mechanikę rozpoznawania ich.

Niemniej zamykanie executora wprowadzone

danius0001 commented 6 years ago

Spoko, to tylko sugestia na przyszłość ;)