[PARQUET-373] MemoryManager tests are flaky

• Type: Bug
• Status: Resolved
• Priority: Major
• Resolution: Fixed
• Affects Version/s: 1.8.0
• Fix Version/s: 1.9.0, 1.8.2
• Component/s: parquet-mr
• Assignee: Ryan Blue
• Reporter: Ryan Blue
• Created: 11/Sep/15 21:57
• Updated: 23/Jun/24 03:27
• Resolved: 19/Oct/15 22:52

Description

The memory manager tests are flaky, depending on the heap allocation for the JVM they run in. This is caused by over-specific tests that assert the memory allocation down to the byte and the fact that some assertions implicitly cast long values to doubles to use the "within" form of assertEquals.

The tests should not validate a specific allocation strategy, but should instead assert that:
1. The allocation for a file is the row group size until room runs out
2. When scaling row groups, the total allocation does not exceed the pool size

Issue Links

• links to: PR #269