damccorm commented on code in PR #23057: URL: https://github.com/apache/beam/pull/23057#discussion_r965076064
########## sdks/go/examples/wordcount/wordcount.go: ########## @@ -95,10 +95,12 @@ var ( // Concept #3: You can make your pipeline assembly code less verbose by // defining your DoFns statically out-of-line. A DoFn can be defined as a Go -// function and is conventionally suffixed "Fn". The argument and return types -// dictate the pipeline shape when used in a ParDo. For example, +// function and is conventionally suffixed "Fn". Using named function +// transforms allows for easy reuse, modular testing, and an improved monitoring +// experience. The argument and return types of a function dictate the pipeline +// shape when used in a ParDo. For example, // -// formatFn: string x int -> string +// func formatFn(w string, c int) string Review Comment: We should make extractFn match below (sorry I didn't call that out initially) -- 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]
