[ 
https://issues.apache.org/jira/browse/FLINK-3234?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15132298#comment-15132298
 ] 

ASF GitHub Bot commented on FLINK-3234:
---------------------------------------

Github user fhueske commented on a diff in the pull request:

    https://github.com/apache/flink/pull/1585#discussion_r51870593
  
    --- Diff: 
flink-java/src/test/java/org/apache/flink/api/java/operator/SortPartitionTest.java
 ---
    @@ -169,6 +169,38 @@ public void testSortPartitionWithExpressionKeys4() {
                tupleDs.sortPartition("f3", Order.ASCENDING);
        }
     
    +   @Test
    +   public void testSortPartitionWithKeySelector1() {
    +           final ExecutionEnvironment env = 
ExecutionEnvironment.getExecutionEnvironment();
    +           DataSet<Tuple4<Integer, Long, CustomType, Long[]>> tupleDs = 
env.fromCollection(tupleWithCustomData, tupleWithCustomInfo);
    +
    +           // should work
    +           try {
    +                   tupleDs.sortPartition(new KeySelector<Tuple4<Integer, 
Long, CustomType, Long[]>, Integer>() {
    +                           @Override
    +                           public Integer getKey(Tuple4<Integer, Long, 
CustomType, Long[]> value) throws Exception {
    +                                   return value.f0;
    --- End diff --
    
    The test only works because the KeySelector function returns the first 
field of the input data. If you change this to `value.f1` the test will fail.


> SortPartition does not support KeySelectorFunctions
> ---------------------------------------------------
>
>                 Key: FLINK-3234
>                 URL: https://issues.apache.org/jira/browse/FLINK-3234
>             Project: Flink
>          Issue Type: Improvement
>          Components: DataSet API
>    Affects Versions: 1.0.0, 0.10.1
>            Reporter: Fabian Hueske
>            Assignee: Chiwan Park
>             Fix For: 1.0.0
>
>
> The following is not supported by the DataSet API:
> {code}
> DataSet<MyObject> data = ...
> DataSet<MyObject> data.sortPartition(
>   new KeySelector<MyObject, Long>() {
>     public Long getKey(MyObject v) {
>       ...
>     }
>   }, 
>   Order.ASCENDING);
> {code}



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

Reply via email to