accumulo-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Sean Busbey" <>
Subject Review Request 19496: ACCUMULO-2487 match Value implementation to javadocs. check for null args.
Date Thu, 20 Mar 2014 21:32:30 GMT

This is an automatically generated e-mail. To reply, visit:

Review request for accumulo.

Bugs: ACCUMULO-2487

Repository: accumulo


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.


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



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.


Sean Busbey

  • Unnamed multipart/alternative (inline, None, 0 bytes)
View raw message