Re: Duplicate sort keys

2016-10-05 Thread Fabian Hueske
Hi Greg, IMO you are right. We should remove duplicate sort keys. Best, Fabian 2016-10-03 16:04 GMT+02:00 Greg Hogan <c...@greghogan.com>: > Is it correct to expect that Flink should remove duplicate sort keys? I'm > working on instrumenting the FixedLengthRecordSorter

Duplicate sort keys

2016-10-03 Thread Greg Hogan
Is it correct to expect that Flink should remove duplicate sort keys? I'm working on instrumenting the FixedLengthRecordSorter (FLINK-4705) and the following test case from TypeHintITCase:200 is having an unexpected effect due to the keyPositions = {0, 0} being passed to TupleComparator. DataSet