[GitHub] [tinkerpop] FlorianHockmann commented on issue #1016: Add request pipelining and a fixed ConnectionPool size TINKERPOP-1775 and TINKERPOP-1774

["FlorianHockmann (GitHub)" <gi...@apache.org>]

Thanks for your comments, @jorgebay. I just pushed an updated version that uses a fixed size for the connection pool and that should also address the other issues you pointed out. The Travis build should now also succeed deterministically.

VOTE +1

edit: I had to remove one test that failed on Travis like 1/10 times. It tried to ensure that a slow request (in the form of a simple `sleep()`) doesn't block another fast request. That failed sometimes, probably because all worker threads of the Gremlin Server were already blocked from other tests that also used these sleeps. I can't think of another way to test this, so I simply removed the test.

[ Full content available at: https://github.com/apache/tinkerpop/pull/1016 ]
This message was relayed via gitbox.apache.org for dev@tinkerpop.apache.org