You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@ignite.apache.org by "Alexey Kuznetsov (JIRA)" <ji...@apache.org> on 2017/08/09 14:21:00 UTC

[jira] [Created] (IGNITE-6013) Web agent: refactor processing response from cluster

Alexey Kuznetsov created IGNITE-6013:
----------------------------------------

             Summary: Web agent: refactor processing response from cluster
                 Key: IGNITE-6013
                 URL: https://issues.apache.org/jira/browse/IGNITE-6013
             Project: Ignite
          Issue Type: Improvement
            Reporter: Alexey Kuznetsov
            Assignee: Alexey Kuznetsov


RestExecutor.sendRequest() contain following code:
{code}
....
        try (Response resp = httpClient.newCall(reqBuilder.build()).execute()) {
            String content = resp.body().string();

            if (resp.isSuccessful()) {
                JsonNode node = mapper.readTree(content);
.... 
{code}

Problems: 
# String content = resp.body().string(); >> Generate not needed String.
# JsonNode node = mapper.readTree(content);  >> Generates a big tree of JsonNodes
 
Could be fixed like:
RestResponseHolder res = MAPPER.readValue(resp.body().byteStream(), RestResponseHolder.class); 

Where for RestResponseHolder should also created optimized RestResponseHolderDeserializer that will extract response as String without building tree of JsonNodes.



--
This message was sent by Atlassian JIRA
(v6.4.14#64029)