accumulo-notifications mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Ryan Leary (JIRA)" <>
Subject [jira] [Updated] (ACCUMULO-1505) MockTable's addMutation does not check for empty mutation
Date Tue, 11 Jun 2013 18:44:21 GMT


Ryan Leary updated ACCUMULO-1505:

    Status: Patch Available  (was: Open)

Added fixes to check for null mutation/mutation iterables as well as empty mutations. Adds
unit test for these cases.
> MockTable's addMutation does not check for empty mutation
> ---------------------------------------------------------
>                 Key: ACCUMULO-1505
>                 URL:
>             Project: Accumulo
>          Issue Type: Bug
>          Components: client
>    Affects Versions: 1.4.3
>            Reporter: Ryan Leary
>             Fix For: 1.4.4, 1.5.1, 1.6.0
> When calling addMutation or addMutations on a MockBatchWriter, the updates stored in
the mutation are iterated over then committed in the MockTable class. 
> When this occurs in the TabletServerBatchWriter (eventually called from the BatchWriterImpl),
however, the mutation size is first checked and if the mutation size is 0, an IllegalArgumentException
is thrown.
> In practice, if you have code that tries to submit an empty mutation to a BatchWriter,
it will fail and throw an exception in the real world, but this will not be caught in tests
against MockAccumulo.

This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see:

View raw message