accumulo-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Bill Havanki" <>
Subject Re: Review Request 16043: ACCUMULO-1959 - deprecate a Value constructor
Date Thu, 05 Dec 2013 20:47:41 GMT

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

(Updated Dec. 5, 2013, 3:47 p.m.)

Review request for accumulo.


Fixed formatting for unit tests and added license header.

Bugs: ACCUMULO-1959

Repository: accumulo


Very minor change, deprecating a constructor for the Value class where a needless double-copy
of the input is performed. I also added a unit test for Value, which really could use one.

Diffs (updated)

  core/src/main/java/org/apache/accumulo/core/data/ 7d3cf8f20aa12205d25027d6ad06e4a3f6062b2e

  core/src/test/java/org/apache/accumulo/core/data/ PRE-CREATION 



Implemented and ran unit test before and after change, all tests pass.


Bill Havanki

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