[CALCITE-5170] Assertion error on range distribution creation

• Type: Bug
• Status: Closed
• Priority: Major
• Resolution: Fixed
• Affects Version/s: 1.30.0
• Fix Version/s: 1.31.0
• Component/s: core
• Labels: pull-request-available
• Assignee: Roman Kondakov
• Reporter: Roman Kondakov
• Created: 29/May/22 09:31
• Updated: 03/Aug/22 16:21
• Resolved: 01/Jun/22 00:32

Description

An assertion error occurs on range distribution creation. This test fails:

@Test void testRangeRelDistributionKeys() {
    RelDistributions.range(Arrays.asList(0, 1));
}

This happens because there is an incorrect assertion in RelDistributionImpl constructor:

      assert type == Type.HASH_DISTRIBUTED
          || type == Type.RANDOM_DISTRIBUTED
          || keys.isEmpty();

It should be RANGE_DISTRIBUTED instead of RANDOM_DISTRIBUTED

Issue Links

• links to: GitHub Pull Request #2822
• links to: GitHub Pull Request #2822