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

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

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

    https://github.com/apache/incubator-flink/pull/194#discussion_r20137955
  
    --- Diff: 
flink-java/src/main/java/org/apache/flink/api/java/operators/JoinOperator.java 
---
    @@ -542,6 +542,33 @@ public void join(IN1 first, IN2 second, Collector<OUT> 
out) throws Exception {
                        return new JoinProjection<I1, I2>(getInput1(), 
getInput2(), getKeys1(), getKeys2(), getJoinHint(), firstFieldIndexes, null);
                }
                
    +           
    +           /**
    +            * Initiates a ProjectJoin transformation and projects the 
first join input<br/>
    +            * If the first join input is a {@link Tuple} {@link DataSet}, 
fields can be selected by their index.
    +            * If the first join input is not a Tuple DataSet, no 
parameters should be passed.<br/>
    +            * 
    +            * Fields of the first and second input can be added by 
chaining the method calls of
    +            * {@link 
org.apache.flink.api.java.operators.JoinOperator.ProjectJoin#projectionFirst(int...)}
 and
    +            * {@link 
org.apache.flink.api.java.operators.JoinOperator.ProjectJoin#projectionSecond(int...)}.
    +            * 
    +            * @param firstFieldIndexes If the first input is a Tuple 
DataSet, the indexes of the selected fields.
    +            *                                         For a non-Tuple 
DataSet, do not provide parameters.
    +            *                                         The order of fields 
in the output tuple is defined by to the order of field indexes.
    +            * @return A ProjectJoin to complete the Join transformation.
    +            * 
    +            * @see Tuple
    +            * @see DataSet
    +            * @see 
org.apache.flink.api.java.operators.JoinOperator.JoinProjection
    +            * @see 
org.apache.flink.api.java.operators.JoinOperator.ProjectJoin
    +            */
    +           public <OUT extends Tuple> ProjectJoin<I1, I2, OUT> 
projectionFirst(int... firstFieldIndexes) {
    --- End diff --
    
    rename to ``projectFirst()`` and ``projectSecond()``.


> Make type() call in projections optional (or remove it)
> -------------------------------------------------------
>
>                 Key: FLINK-1040
>                 URL: https://issues.apache.org/jira/browse/FLINK-1040
>             Project: Flink
>          Issue Type: Improvement
>          Components: Java API
>            Reporter: Stephan Ewen
>            Priority: Minor
>              Labels: simple, starter
>
> I think the type() call should be optional. The compiler can also cast the 
> data set directly and the result type is computed from the input types 
> anyways.



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

Reply via email to