accumulo-notifications mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Christopher Tubbs (JIRA)" <j...@apache.org>
Subject [jira] [Updated] (ACCUMULO-2594) Improve TabletLocator.getInstance() to check tableId points to a table which exists
Date Thu, 18 Dec 2014 18:37:13 GMT

     [ https://issues.apache.org/jira/browse/ACCUMULO-2594?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]

Christopher Tubbs updated ACCUMULO-2594:
----------------------------------------
    Fix Version/s:     (was: 1.6.3)
                       (was: 1.5.3)

This has had no progress in 9 months. Because this does not (appear to) address any bugs using
the public API, and we've adopted semver, I'm dropping patch versions from the fixVersions,
so this improvement should focus on the next minor release.

> Improve TabletLocator.getInstance() to check tableId points to a table which exists
> -----------------------------------------------------------------------------------
>
>                 Key: ACCUMULO-2594
>                 URL: https://issues.apache.org/jira/browse/ACCUMULO-2594
>             Project: Accumulo
>          Issue Type: Improvement
>          Components: client
>    Affects Versions: 1.5.0
>            Reporter: Ara Ebrahimi
>            Priority: Minor
>             Fix For: 1.7.0
>
>
> If you pass an invalid tableId (or in my case tableName instead of tableId) TabletLocator
won't complain.
> TabletLocator tabletLocator = TabletLocator.getInstance(instance, new Text(tableId));
> Next if you use the returned tabletLocator to locate a tablet it just returns an empty
list, never complaining that the tableId was invalid:
> TabletLocator.TabletLocation tabletLocation = tabletLocator.locateTablet(rowKey.getText(),
false, false, credentials.toThrift());
> Ideally TabletLocator.getInstance() should guard against invalid/non-existing tableIds.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

Mime
View raw message