You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@ignite.apache.org by "Rodion (JIRA)" <ji...@apache.org> on 2018/10/30 14:29:00 UTC

[jira] [Assigned] (IGNITE-8896) Wrong javadoc for TcpCommunicationSpi.setSlowClientQueueLimit()

     [ https://issues.apache.org/jira/browse/IGNITE-8896?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Rodion reassigned IGNITE-8896:
------------------------------

    Assignee: Rodion

> Wrong javadoc for TcpCommunicationSpi.setSlowClientQueueLimit()
> ---------------------------------------------------------------
>
>                 Key: IGNITE-8896
>                 URL: https://issues.apache.org/jira/browse/IGNITE-8896
>             Project: Ignite
>          Issue Type: Bug
>    Affects Versions: 2.5
>            Reporter: Dmitry Karachentsev
>            Assignee: Rodion
>            Priority: Minor
>              Labels: newbie
>             Fix For: 2.8
>
>
> Javadoc for TcpCommunicationSpi.setSlowClientQueueLimit() says that is should be set to value equal to TcpCommunicationSpi.getMessageQueueLimit().
> But this is wrong, because first checks back pressure limit and sender thread will be blocked before slow client check. Also, warning checks the value correctly, but message should be "...greater or equal...":
> {code:java}
> if (slowClientQueueLimit > 0 && msgQueueLimit > 0 && slowClientQueueLimit >= msgQueueLimit) {
>             U.quietAndWarn(log, "Slow client queue limit is set to a value greater than message queue limit " +
>                 "(slow client queue limit will have no effect) [msgQueueLimit=" + msgQueueLimit +
>                 ", slowClientQueueLimit=" + slowClientQueueLimit + ']');
>         }
> {code}



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)