[ACCUMULO-2748] MockTableOperations.deleteRow does not handle null for start or end keys

• Type: Bug
• Status: Resolved
• Priority: Critical
• Resolution: Fixed
• Affects Version/s: 1.5.1
• Fix Version/s: 1.5.2, 1.6.0
• Component/s: test
• Labels: noob test

Environment

OSX Mavericks

• Assignee: Mike Fagan
• Reporter: Mike Fagan
• Created: 28/Apr/14 21:50
• Updated: 29/Apr/14 16:45
• Resolved: 29/Apr/14 16:30

Description

The deleteRow function does not check for null values for start or end keys.
These null values are passed down into key constructor which will throw a NullPointerException:
java.lang.NullPointerException
at org.apache.accumulo.core.data.Key.<init>(Key.java:103)
at org.apache.accumulo.core.client.mock.MockTableOperations.deleteRows(MockTableOperations.java:315)

The API semantics dictate:
if (start == null ) then start == Text()
if (end == null ) then end == maxKey()