davidradl commented on code in PR #24618:
URL: https://github.com/apache/flink/pull/24618#discussion_r1582733973


##########
flink-streaming-java/src/main/java/org/apache/flink/streaming/api/transformations/OneInputTransformation.java:
##########
@@ -42,7 +42,7 @@
  * @param <OUT> The type of the elements that result from this {@code 
OneInputTransformation}
  */
 @Internal
-public class OneInputTransformation<IN, OUT> extends 
PhysicalTransformation<OUT> {
+public class OneInputTransformation<IN, OUT> extends 
TransformationWithLineage<OUT> {

Review Comment:
   I can see that some PhysicalTransformations now subclass 
TransformationWithLineage. How did you decide which ones to use? I am thinking 
we would want lineage for MultipleInputTransformation, 
KeyedMultipleInputTransformation, TwoInputTransformation and maybe others?



-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: issues-unsubscr...@flink.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

Reply via email to