[
https://issues.apache.org/jira/browse/SCB-911?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16614474#comment-16614474
]
ASF GitHub Bot commented on SCB-911:
------------------------------------
wujimin commented on a change in pull request #907: [SCB-911]Timeout scenario
print too many logs
URL:
https://github.com/apache/incubator-servicecomb-java-chassis/pull/907#discussion_r217623422
##########
File path:
swagger/swagger-invocation/invocation-core/src/main/java/org/apache/servicecomb/swagger/invocation/exception/ExceptionToResponseConverters.java
##########
@@ -42,8 +44,18 @@ public ExceptionToResponseConverters() {
}
public Response convertExceptionToResponse(SwaggerInvocation
swaggerInvocation, Throwable e) {
- ExceptionToResponseConverter<Throwable> converter =
- exceptionToResponseConverters.getOrDefault(e.getClass(),
defaultConverter);
+ ExceptionToResponseConverter<Throwable> converter = null;
Review comment:
we can cache the result: Map<Class<?>,
ExceptionToResponseConverter<Throwable> >
because the same exception, will always get the same converter
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
> Timeout scenario print too many logs
> ------------------------------------
>
> Key: SCB-911
> URL: https://issues.apache.org/jira/browse/SCB-911
> Project: Apache ServiceComb
> Issue Type: Improvement
> Components: Java-Chassis
> Reporter: liubao
> Assignee: liubao
> Priority: Major
>
> In some scenario, timeout is very normal. customers do not want print too
> much logs in this situation.
--
This message was sent by Atlassian JIRA
(v7.6.3#76005)
