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