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

ASF GitHub Bot commented on APEXCORE-268:
-----------------------------------------

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

    https://github.com/apache/incubator-apex-core/pull/199#discussion_r49212009
  
    --- Diff: 
common/src/main/java/com/datatorrent/common/util/SerializableObject.java ---
    @@ -58,37 +58,29 @@ public Object readResolve() throws ObjectStreamException
           Constructor<? extends SerializableObject> constructor = 
this.getClass().getConstructor(this.getClass());
           try {
             constructor.setAccessible(true);
    -      }
    -      catch (SecurityException ex) {
    +      } catch (SecurityException ex) {
             logger.warn("Accessing copy constructor {} failed.", constructor, 
ex);
           }
           try {
             return constructor.newInstance(this);
    -      }
    -      catch (InstantiationException ex) {
    +      } catch (InstantiationException ex) {
    --- End diff --
    
    Multi exception catch and usage of ReflectiveOperationException.


> Fix existing checkstyle violations in common module
> ---------------------------------------------------
>
>                 Key: APEXCORE-268
>                 URL: https://issues.apache.org/jira/browse/APEXCORE-268
>             Project: Apache Apex Core
>          Issue Type: Sub-task
>            Reporter: Chandni Singh
>            Assignee: Chandni Singh
>




--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

Reply via email to