You are viewing a plain text version of this content. The canonical link for it is here.
Posted to notifications@accumulo.apache.org by "ASF subversion and git services (JIRA)" <ji...@apache.org> on 2014/02/04 21:54:09 UTC

[jira] [Commented] (ACCUMULO-2322) Tablet constructor leaks this, part 2

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

ASF subversion and git services commented on ACCUMULO-2322:
-----------------------------------------------------------

Commit 47cdb92275126ba27157dd8fefa54b30c446c3c8 in branch refs/heads/master from [~bhavanki]
[ https://git-wip-us.apache.org/repos/asf?p=accumulo.git;h=47cdb92 ]

ACCUMULO-2322 Tablet constructor no longer leaks this to recovery process

The method signatures for TabletServer.recover() and TabletServerLogger.recover()
were modified so that the Tablet constructor no longer needs to pass this.


> Tablet constructor leaks this, part 2
> -------------------------------------
>
>                 Key: ACCUMULO-2322
>                 URL: https://issues.apache.org/jira/browse/ACCUMULO-2322
>             Project: Accumulo
>          Issue Type: Bug
>          Components: tserver
>            Reporter: Bill Havanki
>            Assignee: Bill Havanki
>              Labels: this, thread-safety
>
> [~kturner] noted during review of ACCUMULO-1948 that there is a second leaking of this in the {{Tablet}} constructor at the line:
> {code}
> tabletServer.recover(this.tabletServer.getFileSystem(), this, logEntries, absPaths, new MutationReceiver() { ...
> {code}



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