Uploaded image for project: 'Accumulo'
  1. Accumulo
  2. ACCUMULO-1505

MockTable's addMutation does not check for empty mutation

    XMLWordPrintableJSON

Details

    • Bug
    • Status: Resolved
    • Major
    • Resolution: Fixed
    • 1.4.3
    • 1.4.4, 1.5.1, 1.6.0
    • client
    • None

    Description

      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.

      Attachments

        1. accumulo1505.patch
          4 kB
          Ryan Leary

        Activity

          People

            ryanleary Ryan Leary
            ryanleary Ryan Leary
            Votes:
            0 Vote for this issue
            Watchers:
            4 Start watching this issue

            Dates

              Created:
              Updated:
              Resolved: