[ 
https://issues.apache.org/jira/browse/EAGLE-1012?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15992373#comment-15992373
 ] 

ASF GitHub Bot commented on EAGLE-1012:
---------------------------------------

Github user jhsenjaliya commented on a diff in the pull request:

    https://github.com/apache/eagle/pull/925#discussion_r114252915
  
    --- Diff: 
eagle-core/eagle-alert-parent/eagle-alert-service/src/main/java/org/apache/eagle/service/alert/resolver/AttributeResolverFactory.java
 ---
    @@ -43,8 +43,6 @@ public static AttributeResolvable 
getAttributeResolver(String fieldResolverName,
                     fieldResolvableCache.put(fieldResolverName, instance);
                 } catch (ClassNotFoundException e) {
                     throw new AttributeResolveException("Attribute Resolver in 
type of " + fieldResolverName + " is not found", e);
    -            } catch (InstantiationException | IllegalAccessException e) {
    --- End diff --
    
    instead of removing this, it can be used to describe the error, IDE is 
suggesting to collapse the catch block may be because it has identical error 
msg (throw new AttributeResolveException(e)).


> Some language level problems
> ----------------------------
>
>                 Key: EAGLE-1012
>                 URL: https://issues.apache.org/jira/browse/EAGLE-1012
>             Project: Eagle
>          Issue Type: Bug
>          Components: Project Infrastructure
>    Affects Versions: v0.4.0
>         Environment: Linux 2.6.32-279.el6.x86_64 #1 SMP Wed Jun 13 18:24:36 
> EDT 2012 x86_64 x86_64 x86_64 GNU/Linux
>            Reporter: Benedict Jin
>              Labels: JDK
>             Fix For: v0.6.0
>
>         Attachments: Eagle Language Level Problems.txt
>
>   Original Estimate: 168h
>  Remaining Estimate: 168h
>
> Some language level problems
> * Spell
> * Duplicated
> * Lambda
> * Collection
> * String
> * instanceof
> * Complex Method
> * Exception
> Details see attachment files.



--
This message was sent by Atlassian JIRA
(v6.3.15#6346)

Reply via email to