grpc / grpc-java / 2501

82%

master: 89%

push

travis-ci

Close active streams on channelInactive

We think this broke when the stream lifecycle listener was removed.
Observing the stream lifecycle would be the "proper" fix, but it had
notification ordering issues where streams would close before we were
notified of the event that caused the closure, which made it difficult
to provide useful error messages. The ordering of notifications was also
largely undefined.

The long term fix we look forward to is the HTTP/2 child channels, which
should have clearly defined ordering between error notification and
channel closure, and in the order that we need here.

Fixes #1251

5 of 5 new or added lines in 1 file covered. (100.0%)

4460 of 5505 relevant lines covered (81.02%)

0.81 hits per line