[CALCITE-2102] Ignore duplicate ORDER BY keys, and ensure RelCollation contains no duplicates

• Type: Bug
• Status: Closed
• Priority: Major
• Resolution: Fixed
• Fix Version/s: 1.16.0
• Assignee: Julian Hyde
• Reporter: John Fang
• Created: 20/Dec/17 05:44
• Updated: 27/Feb/24 22:23
• Resolved: 26/Dec/17 23:57

Description

I test the query SELECT a, c FROM MyTable ORDER BY a DESC, a DESC. And the collation of LogicalSort is [0 DESC, 0 DESC] , thus it will order by the repeater fields. I think the calcite should removal the repeated fields, thus it make the LogicalSort whose collation is [0 DESC] be more reasonable.
Considering compatibility，it may be better make SqlToRelConverter (or perhaps RelBuilder) ignore sort keys that have appeared earlier in the key. In addition, it should also prevent the incompatible sort keys. It should throw illegalargumentexception for the query SELECT a, c FROM MyTable ORDER BY a DESC, a ASC because of the incompatibility.