Mailing-List: contact notifications-help@accumulo.apache.org; run by ezmlm
Precedence: bulk
Reply-To: jira@apache.org
Date: Fri, 10 Jan 2014 20:47:51 +0000 (UTC)
From: "Eric Newton (JIRA)" <jira@apache.org>
To: notifications@accumulo.apache.org
Message-ID: <JIRA.12687962.1389222349957.78837.1389386871354@arcas>
In-Reply-To: <JIRA.12687962.1389222349957@arcas>
References: <JIRA.12687962.1389222349957@arcas>
Subject: [jira] [Commented] (ACCUMULO-2160) Run findbugs for 1.6
MIME-Version: 1.0
Content-Type: text/plain; charset=utf-8
Content-Transfer-Encoding: 7bit


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

Eric Newton commented on ACCUMULO-2160:
---------------------------------------

bq. Math.random() for Random.nextInt()
I'm guessing that making new instances is cheap enough, but I don't know.

bq. mean that their subclasses can now be considered equal to superinstances
Right, that was the findbugs complaint: that it wouldn't work for subclasses.

bq. Authorizations.contains(String) seems like it could be huge
I'm not sure it changes the behavior at all.

bq. making recentlyUnloadedCache work better
Yep.  I ported it back to 1.4 and 1.5, too.


> Run findbugs for 1.6
> --------------------
>
>                 Key: ACCUMULO-2160
>                 URL: https://issues.apache.org/jira/browse/ACCUMULO-2160
>             Project: Accumulo
>          Issue Type: Task
>            Reporter: Eric Newton
>            Assignee: Eric Newton
>             Fix For: 1.6.0
>
>


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