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

Keith Turner commented on ACCUMULO-2345:
----------------------------------------

it would also be nice to add a comment about the performance reasons for not 
allocation a Violations object off the bat

> Improve ConstraintChecker.check
> -------------------------------
>
>                 Key: ACCUMULO-2345
>                 URL: https://issues.apache.org/jira/browse/ACCUMULO-2345
>             Project: Accumulo
>          Issue Type: Improvement
>          Components: tserver
>    Affects Versions: 1.7.0
>            Reporter: Vikram Srivastava
>            Assignee: Vikram Srivastava
>            Priority: Trivial
>             Fix For: 1.7.0
>
>         Attachments: ACCUMULO-2345.v1.patch.txt
>
>
> The if-else condition inside {{for}} loop can be merged with try-catch since 
> {{throwable}} is not null only if code goes inside the catch section.



--
This message was sent by Atlassian JIRA
(v6.1.5#6160)

Reply via email to