You are viewing a plain text version of this content. The canonical link for it is here.
Posted to notifications@dubbo.apache.org by "khanimteyaz (GitHub)" <gi...@apache.org> on 2019/01/07 16:04:27 UTC

[GitHub] [incubator-dubbo] khanimteyaz commented on pull request #3141: optimize outbound event

Should we refactore this by extrating to a method to make it reusable and more readable e.g. 

```
handler.received(channel,buildErrorResponse((Request)msg);

private static Response buildErrorResponse(Request request) {
   Response response = new Response(request.getId(), request.getVersion());
   response.setStatus(Response.BAD_REQUEST);
   response.setErrorMessage(StringUtils.toString(t));
   return response;
}
```

Is the something do you feel should be good ?

[ Full content available at: https://github.com/apache/incubator-dubbo/pull/3141 ]
This message was relayed via gitbox.apache.org for notifications@dubbo.apache.org