lostluck commented on a change in pull request #15057:
URL: https://github.com/apache/beam/pull/15057#discussion_r662659433
##########
File path: website/www/site/content/en/documentation/programming-guide.md
##########
@@ -675,10 +746,31 @@ the types of the input and output elements. If your
`DoFn` processes incoming
look like this:
{{< /paragraph >}}
+{{< paragraph class="language-go" >}}
+A `DoFn` processes one element at a time from the input `PCollection`. When you
Review comment:
I'm going to push back on this at present.
The lack of func nature is apparent in the example, immediately adjacent to
the text, and shouldn't be over explained right here. At least, not at this
point in the document. There's an entire section on syntactic sugars and
convenience abbreviations for DoFns, which introduces that funcs can be DoFns
too. I've introduced functional DoFns there since they are largely
conveniences, vs the Structural DoFns.
--
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]
