accumulo-notifications mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "ASF subversion and git services (JIRA)" <>
Subject [jira] [Commented] (ACCUMULO-2753) Redundancy in TServerInstance methods
Date Mon, 09 Jun 2014 20:35:01 GMT


ASF subversion and git services commented on ACCUMULO-2753:

Commit ced3293b324331177f743fe0ef866af56c646e50 in accumulo's branch refs/heads/master from
[;h=ced3293 ]

ACCUMULO-2753 used TServerInstance to encode mutations

> Redundancy in TServerInstance methods
> -------------------------------------
>                 Key: ACCUMULO-2753
>                 URL:
>             Project: Accumulo
>          Issue Type: Sub-task
>          Components: tserver
>            Reporter: Bill Havanki
>            Assignee: Eric Newton
>            Priority: Trivial
>              Labels: newbie, refactoring
> While perusing the metadata table code, I noticed that there is a method {{TServerInstance.putFutureLocation(Mutation)}},
but logic in {{MetaDataStateStore.setFutureLocations(Collection<Assignment>)}} doesn't
use it. Either the code should be refactored properly or the {{TServerInstance}} method should
be eliminated.
> From this, the rest of the methods in {{TServerInstance}} should be checked to see if
they are used and useful, and refactoring done as appropriate.

This message was sent by Atlassian JIRA

View raw message