[HIVE-18141] Fix StatsUtils.combineRange to combine intervals

• Type: Sub-task
• Status: Closed
• Priority: Major
• Resolution: Fixed
• Fix Version/s: 3.0.0
• Component/s: Statistics
• Assignee: Zoltan Haindrich
• Reporter: Zoltan Haindrich
• Created: 23/Nov/17 16:49
• Updated: 22/May/18 23:12
• Resolved: 27/Nov/17 14:14

Description

the current combinedRange implementation in its current form "combines" only ranges which contain eachother

but the comments suggests that the intention was to capture the case when the 2 intervals are overlap; can be checked with the following testcase:

  @Test
  public void test11() {
    Range r1 = new Range(0, 1);
    Range r2 = new Range(1, 11);
    Range r3 = StatsUtils.combineRange(r1, r2);
    assertNotNull(r3);
  }