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

Posted to pr@cassandra.apache.org by jeffjirsa <gi...@git.apache.org> on 2017/12/06 00:45:22 UTC

[GitHub] cassandra issue #178: DynamicEndpointSnitch: added a cache for medians.

Github user jeffjirsa commented on the issue:

    https://github.com/apache/cassandra/pull/178
  
    Hi @Benmartin92 , That looks like a perfectly reasonable patch. It appears that @bdeggleston also noticed this in https://issues.apache.org/jira/browse/CASSANDRA-14091 (a week or so after your patch). Unfortunately, I don't think anyone noticed your patch, probably because we don't typically use Github PRs (ASF repo isn't writable on github, so we can't merge this PR directly) - our contribution workflow is through JIRA. Appreciate the contribution, sorry we didn't see it in time. 


---

---------------------------------------------------------------------
To unsubscribe, e-mail: pr-unsubscribe@cassandra.apache.org
For additional commands, e-mail: pr-help@cassandra.apache.org