-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/19496/
-----------------------------------------------------------

Review request for accumulo.


Bugs: ACCUMULO-2487
    https://issues.apache.org/jira/browse/ACCUMULO-2487


Repository: accumulo


Description
-------

Updates Value() to create an empty backing array. Makes methods that touch the 
backing array reject nulls proactively. Changed some internal checks to asserts 
since there ought not be a way to make the backing array null.


Diffs
-----

  core/src/main/java/org/apache/accumulo/core/data/Value.java 39ebbd0 
  core/src/test/java/org/apache/accumulo/core/data/ValueTest.java 9b68222 

Diff: https://reviews.apache.org/r/19496/diff/


Testing
-------

existing and new unit tests pass, with the exception of changing some for the 
default constructor since they matched the previous implementation rather than 
the javadoc.

mvn verify in progress.


Thanks,

Sean Busbey

Reply via email to