[CALCITE-2768] PlannerTest ignores top-level order by clause (~RootRel.collation)

• Type: Test
• Status: Closed
• Priority: Major
• Resolution: Fixed
• Affects Version/s: 1.18.0
• Fix Version/s: 1.19.0
• Component/s: core
• Assignee: Vladimir Sitnikov
• Reporter: Vladimir Sitnikov
• Created: 03/Jan/19 19:13
• Updated: 25/Mar/19 00:10
• Resolved: 03/Jan/19 19:26

Description

    RelTraitSet traitSet = planner.getEmptyTraitSet()
        .replace(EnumerableConvention.INSTANCE);

should be replaced with

    RelTraitSet traitSet = convert.getTraitSet()
        .replace(EnumerableConvention.INSTANCE);

Otherwise PlannerTest asks Planner to produce "empty collation" even in case Root relation has order by.

See also "how does SortRemoveRule work?" in the mailing list.