mizosoft / methanol / #440

89%

Build Type

push

github

Committed by web-flow

Commit Message

ByteChannelBodySubscriber touch-ups (#100)

- Amongst other things, a race issue that lead the channel to read nothing
   when interrupted by another thread instead of throwing is fixed. This was
   caused by the implementation relying on AbstractInterruptibleChannel::end
   to detect interruption even if detected by waiting on BlockingQueue::take.
   This was not reliable as when a thread is interrupted, it first unparks
   parked threads, then calls the channel's interruption hook. This makes
   it possible for the interruption to be detected by the queue, but not
   yet by the channel (AbstractInterruptibleChannel::end). Now the channel
   throws a ClosedByInterruptionException when an InterruptedException is
   thrown by BlockingQueue::take.

 - Another notable change is that an upstream error is logged and not recorded
   if it is found that there is no way for the reader to notice it due to channel
   closure.

Run Details

112 of 117 new or added lines in 3 files covered. (95.73%)

4 existing lines in 2 files now uncovered.

7340 of 8113 relevant lines covered (90.47%)

0.9 hits per line