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

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

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

    https://github.com/apache/incubator-flink/pull/194#discussion_r20142494
  
    --- Diff: 
flink-java/src/main/java/org/apache/flink/api/java/operators/ProjectOperator.java
 ---
    @@ -93,6 +113,30 @@ public Projection(DataSet<T> ds, int[] fieldIndexes) {
                        this.fieldIndexes = fieldIndexes;
                }
                
    +           public void acceptAdditionalIndexes(int... additionalIndexes) {
    --- End diff --
    
    Sure!  ```acceptAdditionalIndexes()``` should be invisible.


> 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