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)