Hi Philip,

thanks for reporting the issue. I just verified the problem.
It is working correctly for the Java API, but is broken in Scala.

I will work on a fix and include it in the next RC for 0.10.0.

Thanks, Fabian

2015-11-02 12:58 GMT+01:00 Philip Lee <philjj...@gmail.com>:

> Thanks for your reply, Stephan.
>
> So you said this is same as SQL
> but I got this result from this code. This is what we did not expect,
> right?
>
> val inputTuple = Seq((2,5),(2,3),(2,4),(3,2),(3,6))
>
> val outputTuple = env.fromCollection(inputTuple)
>   .sortPartition(0,Order.DESCENDING)
>   //.sortPartition(1,Order.ASCENDING)
>   .print()
>
> Output:
> (3,2)
> (3,6)
> (2,5)
> (2,3)
> (2,4)
>
> val inputTuple = Seq((2,5),(2,3),(2,4),(3,2),(3,6))
>
> val outputTuple = env.fromCollection(inputTuple)
>   .sortPartition(0,Order.DESCENDING)
>   .sortPartition(1,Order.ASCENDING)
>   .print()
>
> **
> Actual Output:
> (3,2)
> (2,3)
> (2,4)
> (2,5)
> (3,6)
> ​
> *Expected Output:*
> *(3,2)*
> *(3,6)*
> *(2,3)*
> *(2,4)*
> *(2,5)*
>
>
> Thanks,
> Phil
>
>
> On Mon, Nov 2, 2015 at 5:54 AM, Stephan Ewen <se...@apache.org> wrote:
>
>> Actually, sortPartition(col1).sortPartition(col2) results in a single
>> sort that primarily sorts after col1 and secondarily sorts after col2, so
>> it is the same as in SQL when you state "ORDER BY col1, col2".
>>
>> The SortPartitionOperator created with the first "sortPartition(col1)"
>> call appends further columns, rather than instantiating a new sort.
>>
>> Greetings,
>> Stephan
>>
>>
>> On Sun, Nov 1, 2015 at 11:29 AM, Philip Lee <philjj...@gmail.com> wrote:
>>
>>> Hi,
>>>
>>> I know when applying order by col, it would be
>>> sortPartition(col).setParralism(1)
>>>
>>> What about orderBy two columns more?
>>> If the sql is to state order by col_1, col_2,  sortPartition().
>>> sortPartition () does not solve this SQL.
>>>
>>> because orderby in sql is to sort the fisrt coulmn and the second column
>>> in the sorted first column. but for flink the funtion totally sorts each
>>> column.
>>>
>>> Any suggestion?
>>>
>>> Thanks,
>>> phil
>>>
>>>
>>
>

Reply via email to