accumulo-notifications mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Jacob Meisler (JIRA)" <>
Subject [jira] [Commented] (ACCUMULO-3354) Calling deleteRows on an offline table doesn't include table name in error
Date Tue, 05 May 2015 15:03:01 GMT


Jacob Meisler commented on ACCUMULO-3354:

I was thinking about that.  With the current signature, a caller can pass in a null tableId
to the TableOfflineException constructor and this situation will occur again.

In regards to calling deleteRows specifically, I actually wrote a unit test for this (insert
a few rows, offline the table, delete a row, check the TableOfflineException's message for
the table name).  Unfortunately, MockAccumulo doesn't mock the behavior of offline tables,
so the exception is never thrown.  I could use a MiniAccumuloCluster instead.  Thoughts?

> Calling deleteRows on an offline table doesn't include table name in error
> --------------------------------------------------------------------------
>                 Key: ACCUMULO-3354
>                 URL:
>             Project: Accumulo
>          Issue Type: Bug
>          Components: client
>    Affects Versions: 1.6.1
>            Reporter: Josh Elser
>            Assignee: Jacob Meisler
>              Labels: newbie, summit2015
>             Fix For: 1.7.0, 1.6.3
>         Attachments: ACCUMULO-3354.patch
>          Time Spent: 0.5h
>  Remaining Estimate: 0h
> The table name that I provided to deleteRows should be included in the exception message.
> {noformat}
> org.apache.accumulo.core.client.TableOfflineException: Table  <unknown table> 
is offline
> 	at org.apache.accumulo.core.client.impl.TableOperationsImpl.doFateOperation(
> 	at org.apache.accumulo.core.client.impl.TableOperationsImpl.doFateOperation(
> 	at org.apache.accumulo.core.client.impl.TableOperationsImpl.doTableFateOperation(
> 	at org.apache.accumulo.core.client.impl.TableOperationsImpl.deleteRows(
> {noformat}

This message was sent by Atlassian JIRA

View raw message