accumulo-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Keith Turner <ke...@deenlo.com>
Subject Re: [VOTE] Apache Accumulo 1.6.2 RC1
Date Wed, 21 Jan 2015 23:03:42 GMT
I also ran the compliance checker tool.  The only other changes were in
o.a.a.core.data.KeyValue.  But that class is not listed as part of public
API.  The changes showed up in the report because the class was in data
package.

On Wed, Jan 21, 2015 at 6:01 PM, Christopher <ctubbsii@apache.org> wrote:

> On Wed, Jan 21, 2015 at 11:05 AM, Sean Busbey <busbey@cloudera.com> wrote:
>
> > On Wed, Jan 21, 2015 at 6:57 AM, <dlmarion@comcast.net> wrote:
> >
> > > I concur. This change makes the version of this release 1.7.0. We
> either
> > > need to change the version or remove the method. Good catch. Out of
> > > curiosity, did you find this by visual inspection or with a tool?
> > >
> > >
> > While I have many eyes, they don't generally get spent on comprehensive
> > code reviews. ;)
> >
> > I used the Java API Compatibility Checker.
> >
> >
> >
> Was that the only violation?
>
> (Also, -1 for the same reason.)
>

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