[ https://issues.apache.org/jira/browse/IGNITE-10748?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16764846#comment-16764846 ]
Ignite TC Bot commented on IGNITE-10748: ---------------------------------------- {panel:title=--> Run :: All: No blockers found!|borderStyle=dashed|borderColor=#ccc|titleBGColor=#D6F7C1}{panel} [TeamCity *--> Run :: All* Results|https://ci.ignite.apache.org/viewLog.html?buildId=3047083&buildTypeId=IgniteTests24Java8_RunAll] > Remove dead code in TcpCommunicationSpi for the tcp client creation flow > ------------------------------------------------------------------------ > > Key: IGNITE-10748 > URL: https://issues.apache.org/jira/browse/IGNITE-10748 > Project: Ignite > Issue Type: Task > Reporter: Maxim Muzafarov > Assignee: Maxim Muzafarov > Priority: Minor > Fix For: 2.8 > > Time Spent: 1.5h > Remaining Estimate: 0h > > Currently, a batch of changes can be applied over {{TcpCommunicationSpi}} > class implementation due to: > # {{safeTcpHandshake(..)}} > #* {{handshakeConnIdx}} - parameter is always not null > #* {{recovery}} - parameter is always not null > #* {{HandhakeMessage}} - is not used in current flow at all > # {{HandshakeMessage2}} > #* Message size must be defined in the message class as constant > # {{createTcpClient(..)}} > #* Instantiation of {{GridTcpNioCommunicationClient}} must be performed > outside the while-loop > # Code-style issues. Fix one-liner if statements. > # Extract internal {{HandshakeException}}, {{HandshakeTimeoutException}} from > class. -- This message was sent by Atlassian JIRA (v7.6.3#76005)