You are viewing a plain text version of this content. The canonical link for it is here.

Posted to dev@tinkerpop.apache.org by srondelli <gi...@git.apache.org> on 2018/09/21 22:10:07 UTC

[GitHub] tinkerpop pull request #:

Github user srondelli commented on the pull request:

    https://github.com/apache/tinkerpop/commit/564236ed6aa84682fa5f25d07347d8dfcf73b69a#commitcomment-30598427
  
    Hi Stephen,
    
    There are 2 problems with this solution,
    
    1. It doesn't solve the problem of too many keepAlive requests queued up (and subsequently sent to the Server)
    2. When the queue will get full (which will happen if you have an intense workload with many queries sent every second) you'll start to discard also important jobs.
    
    I'm preparing a pull request based on the fix proposed in the Jira issue.


---