hkeebler commented on issue #1281: Update TabletTime.maxMetadataTime(..) to use MetadataTime objects instead of string URL: https://github.com/apache/accumulo/issues/1281#issuecomment-516420158 the types are checked in the compareTo ```java @Override public int compareTo(MetadataTime mtime) { if (this.type.equals(mtime.getType())) return this.time < mtime.getTime() ? -1 : this.time > mtime.getTime() ? 1 : 0; else throw new IllegalArgumentException( "Cannot compare different time types: " + this + " and " + mtime); } ``` The times do come from the same table. I'm not sure I understand the Long.max() usage here.
---------------------------------------------------------------- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: [email protected] With regards, Apache Git Services
