Re: compile with accumulo 1.7.1
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.
Re: compile with accumulo 1.7.1
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.
compile with accumulo 1.7.1
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.
