[jira] [Commented] (GEODE-8297) TcrMessage default timeout is set to 15ms, rather than 15 seconds

["ASF GitHub Bot (Jira)" <ji...@apache.org>]

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

ASF GitHub Bot commented on GEODE-8297:
---------------------------------------

pdxcodemonkey opened a new pull request #623:
URL: https://github.com/apache/geode-native/pull/623


   


----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


> TcrMessage default timeout is set to 15ms, rather than 15 seconds
> -----------------------------------------------------------------
>
>                 Key: GEODE-8297
>                 URL: https://issues.apache.org/jira/browse/GEODE-8297
>             Project: Geode
>          Issue Type: Bug
>          Components: native client
>            Reporter: Blake Bender
>            Assignee: Blake Bender
>            Priority: Major
>
> While analyzing a log from a bug report, I noticed a trace message from TcrConnection::sendData:
> {quote}{{[debug <<deleted>> 123145454981120] before send len 99 sendTimeoutSec = *15000us* checkConnected = 1 m_connected 1}}
> {quote}
>  
> It appears, from further digging, that we are indeed using a 15ms timeout for (at least) the 
> USER_CREDENTIAL_MESSAGE.  This will happen for any message we don't explicitly set the timeout on before sending.  Potentially this is a cause for a lot of random network timeout failures.  Regardless of whether or not it's for sure causing problems, it's definitely wrong, by 3 orders of magnitude.  This needs to be set to the intended default of 15 _seconds_.
>  



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