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