Abacn commented on code in PR #27868:
URL: https://github.com/apache/beam/pull/27868#discussion_r1298552659
##########
sdks/java/core/src/main/java/org/apache/beam/sdk/transforms/MapElements.java:
##########
@@ -159,6 +159,13 @@ public void processElement(
/** A DoFn implementation that handles a trivial map call. */
private abstract class MapDoFn extends DoFn<InputT, OutputT> {
+
+ /**
+ * Holds {@link MapDoFn#outer} instance of enclosing class, used by {@link
+ * java.lang.reflect.Field} in {@code
SparkNativePipelineVisitor$NativeTransform}.
Review Comment:
No need to refer to `{@code SparkNativePipelineVisitor$NativeTransform}`
here. The comment here could be more general as java core does not depend on
specific runner implementation (and this would give a warning when run
javadoc). Can just say - used by runner implementations.
--
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: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
