[DIRMINA-802] SocketConnector uses non-daemon thread

• Type: Bug
• Status: Closed
• Priority: Minor
• Resolution: Won't Fix
• Affects Version/s: 1.1.7
• Component/s: Transport
• Assignee: Unassigned
• Reporter: Patrick Peck
• Created: 06/Oct/10 09:58
• Updated: 08/Feb/11 10:54
• Resolved: 29/Nov/10 14:44

Description

The no-args constructor of the class org.apache.mina.transport.socket.nio.SocketConnector uses a NewThreadExecutor for handling background processing tasks.

The problem is, that NewThreadExecutor starts a non-daemon thread. This causes problems when the class is used in Tomcat, because the server refuses to shut down and can only be stopped using a process kill.

In my environment, I have fixed the problem by explicitly calling the SocketConnector(int processorCount, Executor executor) constructor and providing an executor that starts a daemon thread.

IMHO, this should be the default behaviour.