accumulo-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Bill Havanki" <>
Subject Re: Review Request 16081: ACCUMULO-1958 - Safer Range constructor
Date Fri, 06 Dec 2013 17:06:52 GMT

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

(Updated Dec. 6, 2013, 12:06 p.m.)

Review request for accumulo.


Resolving Sean's issues.

Bugs: ACCUMULO-1958

Repository: accumulo


Based on Sean's strategy, creates a new protected Range constructor without the start/stop
key check, and adds the check to the public six-argument constructor. Opted not to deprecate
the public constructor at this time, since it is now safe.

Diffs (updated)

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



Unit testing for Range still passes.


Bill Havanki

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