[HIVE-5100] RCFile::sync(long) missing 1 byte in System.arraycopy()

• Type: Bug
• Status: Closed
• Priority: Major
• Resolution: Fixed
• Affects Version/s: 0.12.0
• Fix Version/s: 0.12.0
• Labels: regression
• Release Note: Fix off-by-one regression RCFile::sync(long) resulting in skipped sync points.
• Assignee: Gopal Vijayaraghavan
• Reporter: tagus wang
• Created: 15/Aug/13 01:36
• Updated: 15/Oct/13 23:28
• Resolved: 26/Aug/13 14:34

Description

this has a bug in this:
System.arraycopy(buffer, buffer.length - prefix - 1, buffer, 0, prefix);
it should be
System.arraycopy(buffer, buffer.length - prefix, buffer, 0, prefix);
it is missing 1 byte at the end.