[
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)