Alexey Kukushkin created IGNITE-8067: ----------------------------------------
Summary: Java thin client code review: move parsing from TcpClientCache to ClientMessageParser Key: IGNITE-8067 URL: https://issues.apache.org/jira/browse/IGNITE-8067 Project: Ignite Issue Type: Task Components: thin client Affects Versions: 2.5 Reporter: Alexey Kukushkin Code review comments 9 and 10 from https://issues.apache.org/jira/browse/IGNITE-7421?focusedCommentId=16398310&page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel#comment-16398310 were left as technical debt due to Java thin client delivery urgency. This task is to address: 9) {{SerDes}} - configuration serialization/deserialization is already implemented({{ClientCacheConfigurationSerializer}}). I think all we need is to implement converter from CacheConfiguration to ClientCacheConfiguration and vice versa. 10) {{ClientOperation}} and {{TcpClientCache}} - duplicates logic of {{ClientMessageParser}}. Let's simply add two new methods to decode response and encode request to this class, and remove all lambdas from {{TcpClientCache}} - we should not define the same format twice. -- This message was sent by Atlassian JIRA (v7.6.3#76005)