lostluck commented on a change in pull request #11690:
URL: https://github.com/apache/beam/pull/11690#discussion_r424678091



##########
File path: learning/katas/go/Core Transforms/Map/ParDo Struct/task.md
##########
@@ -18,16 +18,16 @@
 -->
 # ParDo - using a struct as a DoFn
 
-In previous katas, we provided our ParDo a `func` as a DoFn.  In this example, 
we will explore
-the use of a `struct` to in our transform.
+In previous katas, we provided `beam.ParDo` a `func` as a DoFn.  In this 
example, we will explore
+the use of a `struct` to perform our transform.
 
 **Kata:** Implement a simple map function that multiplies all input elements 
by 5.  Use a `struct` as your DoFn with
 a `ProcessElement` method.
 
 <div class="hint">
   Use <a 
href="https://godoc.org/github.com/apache/beam/sdks/go/pkg/beam#ParDo";>
-  ParDo</a>
-  with <a 
href="https://godoc.org/github.com/apache/beam/sdks/go/pkg/beam#hdr-DoFns";>
+  beam.ParDo</a>
+  with a <a 
href="https://godoc.org/github.com/apache/beam/sdks/go/pkg/beam#hdr-DoFns";>
   DoFn</a> as a struct with the ProcessElement method 

Review comment:
       No, as previously pointed out.

##########
File path: learning/katas/go/Core Transforms/Map/ParDo/task.md
##########
@@ -28,8 +28,8 @@ output PCollection.
 
 <div class="hint">
   Use <a 
href="https://godoc.org/github.com/apache/beam/sdks/go/pkg/beam#ParDo";>
-  ParDo</a>
-  with <a 
href="https://godoc.org/github.com/apache/beam/sdks/go/pkg/beam#hdr-DoFns";>
+  beam.ParDo</a>
+  with a <a 
href="https://godoc.org/github.com/apache/beam/sdks/go/pkg/beam#hdr-DoFns";>
   DoFn</a>.

Review comment:
       No, as previously pointed out.




----------------------------------------------------------------
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.

For queries about this service, please contact Infrastructure at:
[email protected]


Reply via email to