[HIVE-7157] skipRows() method in TreeReader subclasses in RecordReaderImpl should check return value from stream.skip()

• Type: Bug
• Status: Resolved
• Priority: Minor
• Resolution: Later
• Assignee: Swarnim Kulkarni
• Reporter: Ted Yu
• Created: 31/May/14 03:03
• Updated: 20/Feb/15 01:06
• Resolved: 20/Feb/15 01:06

Description

This is an example from DoubleTreeReader#skipRows():

      stream.skip(items * 8);

skip() method returns actual number of bytes returned. The return value should be checked against input parameter.

skipRows() method of BinaryTreeReader and StringDirectTreeReader classes has the same issue.