+1; I'd say you made the right call. Matt On Jun 20, 2015 12:48 AM, "Kristian Rosenvold" <krosenv...@apache.org> wrote:
> I reverted the fix in > http://svn.apache.org/viewvc?view=revision&revision=r1686456 simply > because > it broke backward compatibility in a subtle way. The BrokenNNStream classes > supply the same instance of the IOException on all methods (and there are > tests that assert it) > > Given some of the discussions I've seen about backward compatibility here, > I felt it best to revert (even though API-wise I feel this change is within > reasonable even for a minor release). WDYT ? > > Kristian >