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)