accumulo-notifications mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Bill Havanki (JIRA)" <j...@apache.org>
Subject [jira] [Commented] (ACCUMULO-1948) Tablet constructor leaks this
Date Tue, 28 Jan 2014 20:12:39 GMT

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

Bill Havanki commented on ACCUMULO-1948:
----------------------------------------

Third attempt up for review. After testing I found an issue with the simpler second attempt.
Notably, this latest attempt changes the memory management framework for tablets slightly,
so that minor compaction is not attempted on an offline tablet. This makes sense in my limited
understanding of things, so others' opinions are definitely needed. See the review for more
details.

> Tablet constructor leaks this
> -----------------------------
>
>                 Key: ACCUMULO-1948
>                 URL: https://issues.apache.org/jira/browse/ACCUMULO-1948
>             Project: Accumulo
>          Issue Type: Bug
>            Reporter: Keith Turner
>            Assignee: Bill Havanki
>            Priority: Minor
>             Fix For: 1.7.0
>
>
> The tablet constructor leaks this when it calls {{tabletResources.setTablet(this, acuTableConf);}}
 AFAIK this is completey innocuous because nothing is really done (in 1.4 code) w/ the reference.
 Howerver, it is possible the code could be changed in the future to use the ref.



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

Mime
View raw message