"Pipe broken" IOExceptions countered on non-active stream connections

• Key: DIRMINA-129
• Type: Bug
• Status: Closed
• Resolution: Fixed
• Priority: Minor
• Assignee: Trustin Lee
• Reporter: dave irving
• Affects Version/s: 0.8.1
• Fix Version/s: 0.8.2

Description

The StreamIOHandler implementation uses PipedInput/Output streams for providing read data.
PipedInputStream is quite strict about the relationship between the producer and consumer thread.
When blocking on a read, PipedInputStream checks every second to see if the last producer thread to write to it is still alive. If it detects that the producer thread is no longer alive, an IO Exception is thrown indicating "Pipe Broken".

This causes a problem in Mina becuase worker threads in the ThreadPoolFilter have a "keep alive time" (default is one second). So, if a user is blocked on a stream IO read (really a PipedInputStream), and the last ThreadPoolFilter thread which produced data for the stream idles out, the user encounters a "Pipe Broken" IOException (not related to session idle).

Users can currently work around the problem by increasing the keep-alive time on the ThreadPoolFilter.

My gut feeling is that ditching PipedInputStream / PipedOutputStream in favour of something more suited to Minas needs would be a good solution to the problem.

Comments

[Trustin Lee]

This issue affects 0.8, too. We need to fix it in 0.8.2.

I removed the use of PipedInput(Output)Stream in StreamIoHandler. Now, a brand-new and high-performing implementation replaces it. I tested a little bit with HTTP server example, and it seems to work great. Please confirm my fix and close this issue.

Thanks,
Trustin

[dave irving]

Fixed by Trustin