Github user christianbellinaef commented on the pull request:
https://github.com/apache/incubator-tinkerpop/pull/114#issuecomment-149714865
I added test coverage and also applied the changes to the response method.
Mvn clean install, and integration tests both pass.
Looking at the Kryo docs, I see that output.toBytes() gets the [bytes
currently in the output's
*buffer*](http://javadox.com/com.esotericsoftware.kryo/kryo/2.23.0/com/esotericsoftware/kryo/io/Output.html#toBytes())
not the bytes in the underlying output stream being written to.
Calling flush clears the buffer and writes the data, and since we are
currently returning the buffer and not the stream, that would explain the empty
return message.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at [email protected] or file a JIRA ticket
with INFRA.
---
