[IMPALA-11456] Refactor SkipIf logic for filesystems

• Type: Task
• Status: Resolved
• Priority: Minor
• Resolution: Fixed
• Fix Version/s: Impala 4.2.0
• Epic Color: ghx-label-3
• Assignee: Michael Smith
• Reporter: Michael Smith
• Created: 25/Jul/22 17:53
• Updated: 11/Aug/22 17:00
• Resolved: 11/Aug/22 17:00

Description

The SkipIf py.test logic is pretty fragmented. There are cases where it’s not applied consistently, and I can’t tell if those are mistakes or intentional. And it seems far more granular than it needs to be.

Refactor SkipIf classes for the filesystem to use a single class to reduce duplication and inconsistency. If special cases are needed for specific filesystems, those will be called out as different skip cases with their own reasons.