Re: [PR] [FLINK-33211][table] support flink table lineage [flink]

["X-czh (via GitHub)" <gi...@apache.org>]

X-czh commented on code in PR #24618:
URL: https://github.com/apache/flink/pull/24618#discussion_r1644393072


##########
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:
   Looks like `LineageGraphUtils` only handles Source/Sink transformations, why making `OneInputTransformation` extend `TransformationWithLineage`?



-- 
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-unsubscribe@flink.apache.org

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