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/03/05 04:43:44 UTC

[jira] [Commented] (ACCUMULO-2190) Add test for root table file management

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

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

Commit 45d2110edc6717eb6d328b487736e2b8eac44504 in accumulo's branch refs/heads/1.6.0-SNAPSHOT from [~keith_turner]
[ https://git-wip-us.apache.org/repos/asf?p=accumulo.git;h=45d2110 ]

ACCUMULO-2190 Added unit test for root tablet file management code


> Add test for root table file management
> ---------------------------------------
>
>                 Key: ACCUMULO-2190
>                 URL: https://issues.apache.org/jira/browse/ACCUMULO-2190
>             Project: Accumulo
>          Issue Type: Improvement
>            Reporter: Keith Turner
>            Assignee: Keith Turner
>            Priority: Blocker
>             Fix For: 1.6.0
>
>
> The root table manages its own files (the accumulo GC does not delete file for the root table).  It has fault tolerant code for replacing a set of files with a new file after a major compaction.  This code has no test.  The code should be refactored so that it can be easily tested.  I am uncertain if the existing code will work correctly with changes made in ACCUMULO-118.
> The code may be tested with CIAG (continuous ingest and agitation) , but its not guaranteed.  The probability this will be tested with CIAG goes up the longer the test runs.  Since the root tablet is only on one node, the probability of testing this w/ CIAG may go down as more nodes are added and the likely hood of killing the root tablet decreases.  Therefore I think it would be good to have a functional test for this.  Also, since there is uncertantity about ACCUMULO-118 should probably add this for 1.6.0.



--
This message was sent by Atlassian JIRA
(v6.2#6252)