[KYLIN-2871] Ineffective null check in SegmentRange

• Type: Bug
• Status: Closed
• Priority: Minor
• Resolution: Fixed
• Fix Version/s: v2.2.0
• Assignee: Shengping
• Reporter: Ted Yu
• Created: 15/Sep/17 21:22
• Updated: 03/Nov/17 16:48
• Resolved: 24/Sep/17 02:28

Description

For Endpoint#getComparator() :

                    } else if (b.isMax) {
                        return a.isMax ? 0 : -1;
                    } else {
                        if (a == null || b == null)
                            throw new IllegalStateException();

The null check should be placed in the front.