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

Sean Busbey commented on ACCUMULO-2345:
---------------------------------------

Let's just stick with Easymock + Powermock, unless someone wants to make a case 
over on dev@accumulo.

> 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, ACCUMULO-2345.v2.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