Ok, it looks like the two offending usages (at first glance) are ArgumentChecker and AbstractRecordReader. Neither of these are Accumulo public API (nor are they intended to be). The former is easily replaced in Rya with Guava's Preconditions or Objects in JDK7+.

The latter might be a little more difficult, but should really just inherit from the RecordReader in Hadoop. I'll open up a JIRA issue for this.

Josh Elser wrote:
 From the Accumulo side of things, this is very concerning to me.
Anything using the Accumulo public API should work across any
maintenance release (1.7.x).

Maybe this is a sign that Rya is using something it shouldn't be (or
Accumulo needs to expand its definition of public API to include
something else).

Ly, Kiet wrote:
How hard is it to update rya to use accumulo 1.7.x? Develop branch
failed to compile with accumulo 1.7.1.

Confidentiality Notice:: This email, including attachments, may
include non-public, proprietary, confidential or legally privileged
information. If you are not an intended recipient or an authorized
agent of an intended recipient, you are hereby notified that any
dissemination, distribution or copying of the information contained in
or transmitted with this e-mail is unauthorized and strictly
prohibited. If you have received this email in error, please notify
the sender by replying to this message and permanently delete this
e-mail, its attachments, and any copies of it immediately. You should
not retain, copy or use this e-mail or any attachment for any purpose,
nor disclose all or any part of the contents to any other person.
Thank you.

Reply via email to