FlorianHockmann edited a comment on issue #1016: Add request pipelining and a 
fixed ConnectionPool size TINKERPOP-1775 and TINKERPOP-1774
URL: https://github.com/apache/tinkerpop/pull/1016#issuecomment-449062385
 
 
   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.

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
[email protected]


With regards,
Apache Git Services

Reply via email to