[PIG-3066] Fix TestPigRunner in trunk

• Type: Bug
• Status: Closed
• Priority: Major
• Resolution: Fixed
• Affects Version/s: 0.12.0
• Fix Version/s: 0.12.0
• Hadoop Flags: Reviewed
• Assignee: Cheolsoo Park
• Reporter: Cheolsoo Park
• Created: 22/Nov/12 02:37
• Updated: 14/Oct/13 16:46
• Resolved: 27/Dec/12 04:43

Description

This is a regression from PIG-2924. The size of output in PigStats used to be off by 1 byte per store, but that was fixed by PIG-2924. However, two test cases in TestPigRunner wasn't corrected, so they fail in trunk.

The reason why the size of output in PigStats were off by 1 byte per store is because the size variable was initialized by -1 instead of 0.

long bytes = -1;
...
bytes += status.getLen();
...

Issue Links

• is broken by: PIG-2924 PigStats should not be assuming all Storage classes to be file-based storage
• links to: Review Board