ignite-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Alexey Kuznetsov (JIRA)" <j...@apache.org>
Subject [jira] [Created] (IGNITE-6013) Web agent: refactor processing response from cluster
Date Wed, 09 Aug 2017 14:21:00 GMT
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)

Mime
View raw message