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

MockTable's addMutation does not check for empty mutation

    Details

    • Type: Bug
    • Status: Resolved
    • Priority: Major
    • Resolution: Fixed
    • Affects Version/s: 1.4.3
    • Fix Version/s: 1.4.4, 1.5.1, 1.6.0
    • Component/s: client
    • Labels:
      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

          Activity

            People

            • Assignee:
              ryanleary Ryan Leary
              Reporter:
              ryanleary Ryan Leary
            • Votes:
              0 Vote for this issue
              Watchers:
              3 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved: