[YUNIKORN-2153] Move predicateCheckResult type and tests to their own files

• Type: Improvement
• Status: Open
• Priority: Major
• Resolution: Unresolved
• Component/s: core - scheduler
• Target Version: 1.7.0
• Assignee: Vinayak Hegde
• Reporter: Vinayak Hegde
• Created: 14/Nov/23 09:57
• Updated: 09/Sep/24 11:46

Description

The predicateCheckResult type should not be part of the preemption.go file.

All declarations and functions that have a predicateCheckResult as receiver or return value value should be moved. Unused functions should be removed.

Unneeded exports on functions / types should be removed. If exported the comments on the functions and types need to comply with standard go rules.

All remaining functions should get tests to make sure they are working correctly.

Additionally, implement Stringer interface so that zap.Any() calls can be replaced with zap.Stringer() to avoid reflection.

Issue Links

• is related to: YUNIKORN-2152 Move QueuePreemptionSnapshot type and tests to their own files