[jira] [Commented] (GEODE-9380) Replace sleep()s from Nio.SslEngine replace with yields

["ASF subversion and git services (Jira)" <ji...@apache.org>]

    [ https://issues.apache.org/jira/browse/GEODE-9380?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17390124#comment-17390124 ] 

ASF subversion and git services commented on GEODE-9380:
--------------------------------------------------------

Commit b23902aae94c4b86bc06ace919d4fae1026bcb71 in geode's branch refs/heads/master from Kamilla Aslami
[ https://gitbox.apache.org/repos/asf?p=geode.git;h=b23902a ]

GEODE-9380: Replace sleep()s from Nio.SslEngine replace with yields

(cherry picked from commit 1b97c7c25f1fb68a94d5c7008b98630de45d48b6)


> Replace sleep()s from Nio.SslEngine replace with yields
> -------------------------------------------------------
>
>                 Key: GEODE-9380
>                 URL: https://issues.apache.org/jira/browse/GEODE-9380
>             Project: Geode
>          Issue Type: Improvement
>          Components: membership
>            Reporter: Ernest Burghardt
>            Assignee: Kamilla Aslami
>            Priority: Major
>              Labels: pull-request-available
>             Fix For: 1.12.4, 1.13.4, 1.14.0, 1.15.0
>
>
> We noticed a 4-5x increase in the worst-case P2P send latency (most notably sendReplication) in 1.12 vs 1.8. Further analysis showed that the increase in sendReplicationTime is related to how long it takes to create the sender connection. More specifically, all the time spent creating the sender is in the handshake. Experiments with various sleep durations in NioSslEngine.handshake() method showed that this issue can be fixed by replacing both sleeps handshake() method with Thread.yield() calls.



--
This message was sent by Atlassian Jira
(v8.3.4#803005)