TIOStreamTransport.isOpen is false for InputStream or Outpstream only use.

This follows from THRIFT-2530.

TIOStreamTransaport.isOpen changed to be
 

  public boolean isOpen() {
    return inputStream_ != null && outputStream_ != null;
  }

but constructors TIOStreamTransaport(InputStream) and TIOStreamTransaport(OutputStream) leave one of inputStream_ or outputStream_ null.

This makes isOpen false immediately, no close() called. open() does not change the state of object.

 Example:

TIOStreamTransport x1 = new TIOStreamTransport(new ByteArrayInputStream(new byte[1]));
 System.out.println(x1.isOpen());
 TIOStreamTransport x2 = new TIOStreamTransport(new ByteArrayOutputStream());
 System.out.println(x2.isOpen());

is prints false both times.
 
It should be:

  public boolean isOpen() {
    return inputStream_ != null || outputStream_ != null;
  }

or an explicit flag for the open/close state but inputStream_ != null || outputStream_ != null; is enough given the current close implementation.